-
Notifications
You must be signed in to change notification settings - Fork 195
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
feat: mfe file actions with legacy refactor #2059
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, only not approving because the paths differ from our contract. Make sure to update to align to contract or update the contract and communicate changes to @muselesscreator
2c50e26
to
f20269c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, but you have a serializer dupe that needs to be removed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Codecov ReportAll modified lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## feat/2023-dual-ui #2059 +/- ##
====================================================
Coverage ? 94.96%
====================================================
Files ? 189
Lines ? 19433
Branches ? 1766
====================================================
Hits ? 18455
Misses ? 738
Partials ? 240
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
* feat: submission mfe bff endpoints and api * refactor: legacy upload files * refactor: legacy delete file * refactor: legacy file upload url generation * refactor: has any files in upload space * refactor: suffix constants * feat: add mfe file json handlers
* feat: submission mfe bff endpoints and api * refactor: legacy upload files * refactor: legacy delete file * refactor: legacy file upload url generation * refactor: has any files in upload space * refactor: suffix constants * feat: add mfe file json handlers
* feat: submission mfe bff endpoints and api * refactor: legacy upload files * refactor: legacy delete file * refactor: legacy file upload url generation * refactor: has any files in upload space * refactor: suffix constants * feat: add mfe file json handlers
* feat: submission mfe bff endpoints and api * refactor: legacy upload files * refactor: legacy delete file * refactor: legacy file upload url generation * refactor: has any files in upload space * refactor: suffix constants * feat: add mfe file json handlers
TL;DR - File actions refactors and BFF MFE endpoints
JIRA: AU-1391
What changed?
Developer Checklist
FYI: @openedx/content-aurora