-
Notifications
You must be signed in to change notification settings - Fork 7
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.
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
Remove obsolete backward compatibility flag for case search dropdowns #1455
Conversation
Can you remind me why we needed this flag in the first place? After the HQ changes were released and the remaining FP changes were done, HQ and Formplayer would have both working with keys right? What's the transitory period where FP still needed to support indexes? Is it the transient period where a user is already in a form and then the "key" support changes of Formplayer gets deployed? |
Yes the transient period was very short only to support requests in close proximity of deploy (When user is already on case search screen before and after the FP deploy). |
@Jtang-1 Flagging that this flag can be removed from Web Apps as well once this gets deployed. |
Required validation test fixes
Codecov Report
@@ Coverage Diff @@
## master #1455 +/- ##
============================================
- Coverage 69.74% 69.66% -0.08%
+ Complexity 1922 1918 -4
============================================
Files 246 246
Lines 7546 7543 -3
Branches 676 676
============================================
- Hits 5263 5255 -8
- Misses 2019 2022 +3
- Partials 264 266 +2
... and 1 file with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Technical Summary
remove backward compatibility flag introduced as part of the spec as all response to FP should have it set to
true
now.Safety Assurance
Safety story
Removes a obsolete flag
tested on staging.
Automated test coverage
Updated tests in the PR to test for new behaviour
QA Plan
No QA
Special deploy instructions
Rollback instructions
Review
cross-request: dimagi/commcare-core#1325