Reorder case properties for bulk case export #34743
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Product Description
Just a follow up to #34502 to implement order for bulk case exports.
For bulk case exports, when a new bulk export is created or when a bulk export is edited & saved, ordering of columns is set based on data dictionary.
This is different from how it was implemented earlier for exports for a single case type where the ordering was set only while creating the export and not during edit.
This is because, in case exports for a single case type, users can see columns & reorder them on the UI whereas in bulk case export users don't see the columns on the UI.
So, it is safe to reorder columns based on data dictionary for bulk case exports for new and the ones being edited whereas for case exports for single case type, implementing ordering was only done for new exports and not on edit to avoid removing any re-ordering done by users.
This change is limited to projects with access to data dictionary only.
No screenshots attached for reference, simply the excel sheet is populated as ordered columns based on data dictionary set at the time of creation/edit.
Technical Summary
https://dimagi.atlassian.net/browse/SC-3675
Feature Flag
Safety Assurance
Safety story
Tested locally that things work okay for bulk case exports.
Code changes are quite limited and should not impact anything about exports otherwise.
Automated test coverage
Added tests for new code added.
QA Plan
Will go through QA considering this is a user facing change.
QA request submitted. https://dimagi.atlassian.net/browse/QA-6656
Rollback instructions
Labels & Review