Skip to content

Ush 1415 fix #1417

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

Merged
merged 6 commits into from
Nov 5, 2024
Merged

Ush 1415 fix #1417

merged 6 commits into from
Nov 5, 2024

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
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.

2 participants