-
-
Notifications
You must be signed in to change notification settings - Fork 230
ExportInstance API #37223
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
base: master
Are you sure you want to change the base?
ExportInstance API #37223
Conversation
corehq/apps/api/resources/v1_0.py
Outdated
| form_exports = [ | ||
| FormExportInstance.wrap(result['doc']) | ||
| for result in form_results | ||
| if result['doc'].get('show_det_config_download', False) | ||
| ] |
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
|
||
| class Meta(CustomResourceMeta): | ||
| resource_name = 'det_export_instance' | ||
| authentication = RequirePermissionAuthentication(HqPermissions.view_reports) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
view_reports seems like the right permission to be checking, but please let me know if you think there is a better one to use.
Technical Summary
🦖 SAAS-18882
This adds an API for
ExportInstanceobjects that haveshow_det_config_download= True.This is required by CommCare Data Pipeline (formerly CommCare Sync) to list the available exports on HQ, and download the DET config file for the CommCare Data Pipeline user.
Safety Assurance
Safety story
Automated test coverage
Yes
QA Plan
CommCare Data Pipeline changes will go through QA, including DET config file download.
Rollback instructions
Labels & Review