Skip to content
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

Refactor submission photos #2189

Conversation

Anuj-Gupta4
Copy link
Collaborator

What type of PR is this? (check all applicable)

  • πŸ• Feature
  • πŸ› Bug Fix
  • πŸ“ Documentation
  • πŸ§‘β€πŸ’» Refactor
  • βœ… Test
  • πŸ€– Build or CI
  • ❓ Other (please specify)

Related Issue

Related to
#1875
#1701
will impact
#1706

Describe this PR

  • Configure Central so submission media automatically syncs to S3 (24hr schedule).
  • Optionally trigger the sync on a more frequent interval (perhaps whenever submissions are requested).
  • Implement some additional async functions in osm-fieldwork OdkCentralAsync to handle getting all pre-signed URLs for media attached to a submission.
  • Replace custom logic in backend API with logic from osm-fieldwork to get the submission media pre-signed URLs via S3.
  • Return the pre-signed URLs to the frontend for display.
  • Deploy ODK Central on prod with env vars and S3 bucket set up
  • Remove logic from FMTM that handled upload to the fmtm-data S3 bucket (replaced by ODK logic)
  • Remove photo uploads added fmtm-data bucket on prod

@github-actions github-actions bot added enhancement New feature or request backend Related to backend code ODK Any requests for optimizing ODK labels Feb 13, 2025
@Anuj-Gupta4 Anuj-Gupta4 changed the title Feat/central s3 media tokha Refactor submission photos Feb 13, 2025
@Anuj-Gupta4 Anuj-Gupta4 requested a review from Sujanadh February 13, 2025 10:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Related to backend code enhancement New feature or request ODK Any requests for optimizing ODK
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants