- Issue created by @daffie
- Assigned to nishtha.pradhan
- Issue was unassigned.
- Merge request !9536Added the new method and changed core use to the new methods → (Open) created by daffie
- Status changed to Needs review
about 2 months ago 9:13am 19 September 2024 - Status changed to Needs work
about 2 months ago 1:02pm 19 September 2024 - 🇺🇸United States smustgrave
Left comments on MR. Probably need a CR now too.
Is there anyway to capture or throw exception if someone uses addExpression incorrectly?
- Status changed to Needs review
about 2 months ago 7:12am 20 September 2024 - Status changed to RTBC
about 2 months ago 12:46am 21 September 2024 - 🇳🇿New Zealand quietone
I read through this and there was a question asked in #8 that I can not find an answer to. I added that to the remaining tasks and am setting back to needs work.
So this is adding 9 wrappers but the documentation does not help the developer to know when to use them. Should they continue to use addExpression or one of the wrappers? How would they choose? I do think this needs to be made clear in the docs and in the change record.