OrchardCore icon indicating copy to clipboard operation
OrchardCore copied to clipboard

#10849 Liquid syntax support for CorrelateTask

Open lampersky opened this issue 1 year ago • 3 comments

fixes #10849

this is how it works:

https://user-images.githubusercontent.com/6403130/179355188-7f62d94a-532b-4e02-b18d-9a1e45772c2e.mp4

/cc @Skrypt

lampersky avatar Jul 16 '22 12:07 lampersky

Seems there's a breaking changes

hishamco avatar Jul 16 '22 13:07 hishamco

Seems there's a breaking changes

@hishamco no, there is not

here you have recipe from OC before that PR, and you can test it with this PR, and see that, this feature has backward compatibility

Recipe.zip

lampersky avatar Jul 16 '22 13:07 lampersky

This pull request has merge conflicts. Please resolve those before requesting a review.

github-actions[bot] avatar Jan 29 '24 13:01 github-actions[bot]

@piedone maybe we should take a look more often to PR marked with "need triage" on thursday meetings. I've been trying to make @sebastienros to take a look at those by marking these with that tag. I think I had no comment to add on the PR and it just needed triage.

Skrypt avatar Mar 14 '24 12:03 Skrypt

Sure, please bring them up. I think we only need to triage PRs on the meeting when there's something to discuss with the wider audience (this PR is like that, then?), i.e. you as a reviewer aren't sure.

Please keep to these @lampersky:

  • Apply code suggestions directly so the reviewer doesn't have to eyeball the changes. These resolve themselves after applying them, that's fine.
  • Don't resolve other conversations so it's easier to track for the reviewer. Then, the reviewer will resolve them.
  • Feel free to mark conversations that you addressed to keep track of them with an emoji or otherwise, just don't resolve them.
  • Please keep conversations happening in line comment in those convos, otherwise communication will be a mess. If you have trouble finding them, see this video.
  • Please click "Re-request review" in the top-right corner for each reviewer when you're ready for another round of review, so they know that you're done.

Piedone avatar Mar 14 '24 13:03 Piedone

When you're done addressing all feedback of a review, click "Re-request review" in the top-right corner for each reviewer when you're ready for another round of review, so they know that you're done.

Piedone avatar Apr 10 '24 16:04 Piedone