fix(#10183): needs_signoff string bug

[jonathanbataire]

Description

Current implementation assumes that the report/doc value produced from the xlsform is type boolean true/false yet all values in the doc are string literals
As a result if(doc.fields.needs_signoff) will always be true as long as a key and value exists in the doc with no regards to the value.

changes

The change here is to handle the needs_signoff value as a string

Fixes #10183

Code review checklist

• UI/UX backwards compatible: Test it works for the new design (enabled by default). And test it works in the old design, enable can_view_old_navigation permission to see the old design. Test it has appropriate design for RTL languages.
• Readable: Concise, well named, follows the style guide, documented if necessary.
• Documented: Configuration and user documentation on cht-docs
• Tested: Unit and/or e2e where appropriate
• Internationalised: All user facing text
• Backwards compatible: Works with existing data and configuration or includes a migration. Any breaking changes documented in the release notes.

License

The software is provided under AGPL-3.0. Contributions to this project are accepted under the same license.