Skip to content

Ush 1588 - View Multimedia Content#1384

Merged
ushahidlee merged 24 commits intodevelopmentfrom
ush-1415
Oct 29, 2024
Merged

Ush 1588 - View Multimedia Content#1384
ushahidlee merged 24 commits intodevelopmentfrom
ush-1415

Conversation

@ushahidlee
Copy link
Contributor

@ushahidlee ushahidlee commented Oct 20, 2024

Issues:

Users should be able to create posts with multimedia content: images, audio and documents.

Solution:

This PR introduces 3 new field types: image, document and audio, and a new form component, Media Uploader, to handle the management of that content.

Testing:

Surveys

  • Can be created with new fields.

Posts

  • Can be edited with new fields.
  • Can be viewed with new fields.

Uploader Component

  • Can be configured to limit the upload size and will prevent users from exceeding that limit.
  • Can handle multiple files being uploaded at once (a selection being entered in the system dialog)
  • Allows the users to select from the required filetypes,
    • Image: png, jpeg, gif
    • Document: pdf, doc, docx
    • Audio: mp3, ogg, aac
  • Handles errors elegantly, preventing upload limits from being exceeded and allowing a user to remove offending files from a larger list to submit the post.

Tickets:

https://linear.app/ushahidi/issue/USH-1415/optimize-multimedia-uploads-web-client
https://linear.app/ushahidi/issue/USH-1588/view-multimedia-uploads-web

@ushahidlee ushahidlee requested a review from Angamanga October 20, 2024 15:05
@linear
Copy link

linear bot commented Oct 20, 2024

Copy link
Contributor

@Angamanga Angamanga left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All good now! 🥇

@ushahidlee ushahidlee merged commit 1db0a97 into development Oct 29, 2024
}
this.onChange(this.mediaFiles);
inputElement.value = '';
console.log(results);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ushahidlee 👋 pointing at this console.log() just in case you didn't have the intention to leave it

tuxpiper pushed a commit that referenced this pull request Nov 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants