You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Alex: Another issue is that if the user enters the Error Code with quotes, which is our standard convention, it does not work. But does without quotes, which our standard convention says should be interpreted as a variable.
KB: yeah, that's confusing. i assume in this case there is no option to evaluate it.
Elizabeth Esswein: this field is always a string. So in fact, quotes would be misleading. People already get bitten when they put in a number and expect to be able to compare it to an int. Having quotes introduces the expectation that there could be other types besides strings.
KB: i wonder if we could just strip them out if you type them. first i thought helpful error message, but then i was like, that's annoying
Elizabeth Esswein: Stripping would be a viable option i think
The text was updated successfully, but these errors were encountered:
burnettk
changed the title
strip out quotes when user types them for error boundary event code
strip out quotes when user types them for error boundary error code
May 8, 2024
Alex: Another issue is that if the user enters the Error Code with quotes, which is our standard convention, it does not work. But does without quotes, which our standard convention says should be interpreted as a variable.
KB: yeah, that's confusing. i assume in this case there is no option to evaluate it.
Elizabeth Esswein: this field is always a string. So in fact, quotes would be misleading. People already get bitten when they put in a number and expect to be able to compare it to an int. Having quotes introduces the expectation that there could be other types besides strings.
KB: i wonder if we could just strip them out if you type them. first i thought helpful error message, but then i was like, that's annoying
Elizabeth Esswein: Stripping would be a viable option i think
The text was updated successfully, but these errors were encountered: