Skip to content

Conversation

@ushahidlee
Copy link
Contributor

Issues:

The functionality (described in USH-1415) initially committed had 2 bugs:

  • Mime types were not strictly enforced in the frontend.
  • An upload that failed due to size, could not be removed.
  • Editing uploads exploded.

Solution:

This PR contains:

  • fixes for all of the above issues.
  • much stronger type safety with a class replacing the previous type
  • More enums!
  • Moves the MediaFile class and its associated Enums into the shared library to be used in mobile.

Testing:

USH-1415 functionality but with mime types being enforced and the ability to remove failed uploads.

- Greatly improved type safety
- Deleting failed uploads
- Validation of mime types
- Enums!
- Replaced the type with a class
@ushahidlee ushahidlee requested a review from Angamanga November 1, 2024 20:22
@linear
Copy link

linear bot commented Nov 1, 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.

Looks good 🙌

@ushahidlee ushahidlee merged commit 7aa8e5c into development Nov 5, 2024
10 of 12 checks passed
tuxpiper pushed a commit that referenced this pull request Nov 10, 2024
* Fixes
- Greatly improved type safety
- Deleting failed uploads
- Validation of mime types
- Enums!
- Replaced the type with a class

* fix long filenames
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