-
Notifications
You must be signed in to change notification settings - Fork 0
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
Uppy refresh token logic #1152
Uppy refresh token logic #1152
Conversation
- Fix existing code so it use relevant api client instead of imsApi - refactor thumbnail view logic to use displayed images - remove upload error uppy event listener as it not needed as backend deals with image errors - create uppy uppyOnBeforeRequest function that set the auth headers - create uppyOnAfterResponse to deal with 403 errors
Codecov ReportAttention: Patch coverage is
✅ All tests successful. No failed tests found.
Additional details and impacted files@@ Coverage Diff @@
## develop #1152 +/- ##
===========================================
- Coverage 99.10% 98.96% -0.15%
===========================================
Files 94 94
Lines 17527 17563 +36
Branches 2978 2981 +3
===========================================
+ Hits 17370 17381 +11
- Misses 153 178 +25
Partials 4 4 ☔ View full report in Codecov by Sentry. |
remove timeout |
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.
Worked well for me.
Description
Testing instructions
Agile board tracking
closes #1147