-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
fix: LSDV-4711: CORS error accessing previously annotated data #4091
Conversation
✅ Deploy Preview for heartex-docs canceled.
|
✅ Deploy Preview for label-studio-docs-new-theme canceled.
|
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## develop #4091 +/- ##
===========================================
+ Coverage 76.63% 76.68% +0.04%
===========================================
Files 155 155
Lines 11668 11669 +1
===========================================
+ Hits 8942 8948 +6
+ Misses 2726 2721 -5 see 4 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
* fix: LSDV-4711: CORS error accessing previously annotated data * convert to tavern tests * reinstating the presign_storage_data test as it covers directly the functions * fix the resolving uri in the tests
…Signal#4091) * fix: LSDV-4711: CORS error accessing previously annotated data * convert to tavern tests * reinstating the presign_storage_data test as it covers directly the functions * fix the resolving uri in the tests
PR fulfills these requirements
[fix|feat|ci|chore|doc]: TICKET-ID: Short description of change made
ex.fix: DEV-XXXX: Removed inconsistent code usage causing intermittent errors
Change has impacts in these area(s)
(check all that apply)
Describe the reason for change
Follow up adjustments to fix some consistency issues, and a possible error return based on code review.
Does this PR introduce a breaking change?
(check only one)
What level of testing was included in the change?
(check all that apply)
Which logical domain(s) does this change affect?
Presigned Urls, CORS