[ ] PR title descriptive (can be used in release notes)
Description
Targets branch of pr #2307
add generic parseStringValue string util and tests to cover conversion of string to boolean, number, null or undefined
update appStringEvaluator to no longer parse boolean strings, instead applying new method only to appendColumns operator
Dev notes
I couldn't decide between making the conversion more specific to the appendColumns operator or generally to the appStringEvaluator, but in the end felt keeping it more specific for now might be better as fewer potential knock-ons and can consider making more general in a follow-up PR if useful.
Git Issues
Closes #
Screenshots/Videos
If useful, provide screenshot or capture to highlight main changes
PR Checklist
Description
Targets branch of pr #2307
parseStringValue
string util and tests to cover conversion of string to boolean, number, null or undefinedappStringEvaluator
to no longer parse boolean strings, instead applying new method only toappendColumns
operatorDev notes
I couldn't decide between making the conversion more specific to the
appendColumns
operator or generally to theappStringEvaluator
, but in the end felt keeping it more specific for now might be better as fewer potential knock-ons and can consider making more general in a follow-up PR if useful.Git Issues
Closes #
Screenshots/Videos
If useful, provide screenshot or capture to highlight main changes