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

Jump to bottom

Merge post media values into the posts API itself #2393

Open
rjmackay opened this issue Jan 9, 2018 · 0 comments
Open

Merge post media values into the posts API itself #2393

rjmackay opened this issue Jan 9, 2018 · 0 comments
Labels
Codebase: API Indicates issue work will be in API Tech Debt Theme: Page load time and performance
Milestone
zz-archived

Comments

@rjmackay
Copy link
Contributor

rjmackay commented Jan 9, 2018 •
edited by rowasc

Current behaviour:

To attach media to a post, first we upload media to the media API, then we push the media_id to the posts API

New behaviour:

  • Attach media to a post by including the media blob in the posts JSON and posting to the posts API (See rollcall avatar uploads)
  • Return media URLs in post.values from the posts API
  • Keep existing media API for backwards compatibility, but make as deprecated.

Follow up: Do the same for site logos

Aha! Link: https://ushahiditeam.aha.io/features/PROD-452
@zube zube bot removed the Backlog label Jan 29, 2018
@kinstella kinstella self-assigned this Apr 10, 2018
@rowasc rowasc added the Codebase: API Indicates issue work will be in API label Jun 11, 2019
@rowasc rowasc added this to the Parking Lot milestone Sep 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Codebase: API Indicates issue work will be in API Tech Debt Theme: Page load time and performance
Projects
None yet
Milestone
zz-archived
Development

No branches or pull requests
4 participants
@rjmackay @rowasc @jshorland @kinstella