Skip to content

Master to formplayer (2.60) #1506

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
shubham1g5 wants to merge 567 commits into
base: formplayer
Choose a base branch
from
Open

Master to formplayer (2.60) #1506

shubham1g5 wants to merge 567 commits into from

Conversation

@shubham1g5
Copy link
Contributor

@shubham1g5 shubham1g5 commented Oct 16, 2025 •
edited
Loading

Reconcilling Commcare 2.60 with Formplayer

Safety story

Relying on Formplayer extensive tests coverage to catch any regressions here.

Special deploy instructions

  • This PR can be deployed after merge with no further considerations.

Rollback instructions

  • This PR can be reverted after deploy with no further considerations.

Review

  • The set of people pinged as reviewers is appropriate for the level of risk of the change.

Duplicate PR

Automatically duplicate this PR as defined in contributing.md.

Jtang-1 and others added 30 commits December 11, 2023 17:19
@Jtang-1
treat string of just blank spaces as empty string
1101c59
@shubham1g5
Merge pull request #1391 from dimagi/copy_of_jt/post_data_blank_fix
0c57c80 
Duplicate of jt/post_data_blank_fix
@Jtang-1
feat: parses group_key from <prompt>
dabe4dc
@Jtang-1
feat: parses group element from suite
5407918
@Jtang-1
feat: serialization and deserialization support for the userQueryGrou…
36e984d 
…pHeaders field.
@Jtang-1
feat: adds a groupHeaders param to query definition
9a27952
@Jtang-1
test: group_key attribute of prompt element is parsed correctly
c40aeeb
@Jtang-1
test: group element is parsed and contains key
b37cacc
@Jtang-1
style: correct variable name
34931ff
@Jtang-1
refactor: groupHeaders do not need to be ordered. It will be used by …
6ca4317 
…queryPrompts so utillizes order of queryPrompts
@Jtang-1
style: use constant variable
f7c407f
@Jtang-1
refactor: use key that QueryGroupParser already parsed
9e16254
@Jtang-1
feat: validate group block contains expected elements/attributes
6756312
@Jtang-1
minor style + refactor
43e1d49
@Jtang-1
test: adds display block to group element to pass validation
b05a230
@Jtang-1
refactor: moves evaluation of groupHeader text to QueryScreen
8c3f0e9
@shubham1g5
Merge pull request #1392 from dimagi/copy_of_jt/visually-segmented-ca…
544393c 
…se-search-group

Duplicate of jt/visually-segmented-case-search-group
@Robert-Costello
Add flag to show different shading on case tile fields
eee2acc
@Robert-Costello
add test
8d8d056 
cherry pick resolve conflict
@shubham1g5
Merge pull request #1394 from dimagi/copy_of_rc/field-shading
eaf846c 
Copy of rc/field shading
@Jtang-1
feat: parses for searchOnClear in suite
81dc60e
@Jtang-1
feat: populates remote query session with searchOnClear value
4d76133
@Jtang-1
feat: populates queryScreen with searchOnClear value
a118ac7
@Jtang-1
test: add tests that datum contains search_on_clear value as defined …
3ca9cb5 
…in suite
@shubham1g5 @avazirna
update property names for gradle 8 apis
7072ce9
@shubham1g5 @avazirna
update gradle version for git actions
2eb98dc
@shubham1g5 @avazirna
remove old fix for Jenkins test as we are using Github Actions and gr…
843bb42 
…adle now
@shubham1g5 @avazirna
update gradle version for git actions
3c3f73d
@shubham1g5
Merge pull request #1398 from dimagi/copy_of_jt/clear-on-search
a4dfff4 
Duplicate of jt/clear-on-search
@nospame @shubham1g5
Parse alt_text node and add to DetailField
ae2f7be
shubham1g5 and others added 10 commits September 19, 2025 17:25
@coderabbitai
Copy link

coderabbitai bot commented Oct 16, 2025

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

🗂️ Base branches to auto review (1)
  • commcare_*

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch masterToFormplayer2.60

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@shubham1g5 shubham1g5 mentioned this pull request Oct 16, 2025
Draft
4 tasks
@shubham1g5 shubham1g5 requested a review from avazirna October 16, 2025 16:00
@shubham1g5 shubham1g5 marked this pull request as ready for review October 16, 2025 16:00
@shubham1g5 shubham1g5 mentioned this pull request Oct 28, 2025
Merged
@shubham1g5 shubham1g5 mentioned this pull request Oct 31, 2025
Merged
3 tasks
jjackson and others added 11 commits October 31, 2025 13:00
@jjackson
Fix CLI hanging on exceptions
a349e4e 
Add try-catch block around cliCommand.handle() to ensure the CLI
exits cleanly with error code -1 when exceptions occur, rather than
hanging indefinitely.

This improves the development experience when using CLI tools like
'validate' for CCZ validation, allowing for faster iteration.
@shubham1g5
Apply suggestion from @shubham1g5
273342c
@shubham1g5
Correct escape char
8a452d0
@shubham1g5
Merge pull request #1509 from jjackson/fix/cli-exit-on-error
89627d6 
Fix CLI hanging on exceptions
@shubham1g5
Merge pull request #1508 from dimagi/removeDuplicateEntityScreenChange
926cab9 
removes duplicate entity skip change
@shubham1g5
adds argument validations for auth classes
1d3c02c
@shubham1g5
Merge pull request #1511 from dimagi/addAuthValidations
c2e1774 
Adds argument validations for auth classes
@avazirna
Merge pull request #1512 from dimagi/commcare_2.61
ca085e9 
Backmerge Commcare 2.61.1
@OrangeAndGreen
Added createPointList helper method to GeoPointUtils.
b660fd8 
Using the new helper in PolygonUtils.createPolygon.
Moved validateCoordinates from PolygonUtils to GeoPointUtils.
@OrangeAndGreen
Merge pull request #1513 from dimagi/map_point_list
7eccc74 
Added createPointList helper method to GeoPointUtils
@shubham1g5
Merge branch 'master' into masterToFormplayer2.60
5ce1d38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@avazirna avazirna avazirna approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.