-
Notifications
You must be signed in to change notification settings - Fork 26
feat(form): add case infos to jexl context #2703
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
a06af60
to
8e887d1
Compare
8e887d1
to
885634c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just one question.
Don't forget to rename the property once a decision is made but I will approve now. |
f42c6f3
to
b184c94
Compare
But the naming between the backend and the frontend doesn't match, does it? In the backend we now have a Should we try to keep the naming consistent between frontend and backend? |
The backend MR was merged a little bit too quickly, here is the follow-up which should line up with the frontend part: projectcaluma/caluma#2205 |
b184c94
to
dfa25c8
Compare
dfa25c8
to
a7ec0a1
Compare
@anehx I added another test for unlinked documents. Want to take another look? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💯 🚀
🎉 This PR is included in version 12.15.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
This is convenient when you'd like to write a JEXL expression in a task form attached to some work item, that depends on the case's main documents' form.
Related backend MRs: