javarosa icon indicating copy to clipboard operation
javarosa copied to clipboard

Fixed using raw strings as requiredMsg

Open grzesiek2010 opened this issue 9 months ago • 0 comments

Closes #730

What has been done to verify that this works as intended?

I've tested the changes manually with ODK Collect and also added automated tests.

Why is this the best possible solution? Were any other approaches considered?

Nothing to discuss here.

How does this change affect users? Describe intentional changes to behavior and behavior that could have accidentally been affected by code changes. In other words, what are the regression risks?

We can focus on testing custom retired messages. The fix is rather safe and isolated so I can't think of any risk in any other area of the project. However we should start testing once we implement changes in Collect and start using the new method for reading required messages.

Do we need any specific form for testing your changes? If so, please attach one.

I used the one attached in the issue.

Does this change require updates to documentation? If so, please file an issue here and include the link below.

No.

grzesiek2010 avatar Sep 27 '23 11:09 grzesiek2010