Skip to content
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

Hotfix for file-overwriting from uppy #942

Merged
merged 1 commit into from
Oct 18, 2024
Merged

Conversation

ml-evs
Copy link
Member

@ml-evs ml-evs commented Oct 17, 2024

Do not consider filenames when uploading a file via uppy, just show the alert

Closes #939

@ml-evs ml-evs added the bug Something isn't working label Oct 17, 2024
Copy link

codecov bot commented Oct 17, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 68.38%. Comparing base (21ed72d) to head (8bf56df).
Report is 81 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #942   +/-   ##
=======================================
  Coverage   68.38%   68.38%           
=======================================
  Files          62       62           
  Lines        3928     3928           
=======================================
  Hits         2686     2686           
  Misses       1242     1242           

Copy link

cypress bot commented Oct 18, 2024

datalab    Run #2594

Run Properties:  status check passed Passed #2594  •  git commit 71c344702b ℹ️: Merge 8bf56df2336fadf58e98672cdbfbeb913d304733 into 21ed72d35f6b26d3670d1bfba22a...
Project datalab
Run status status check passed Passed #2594
Run duration 06m 25s
Commit git commit 71c344702b ℹ️: Merge 8bf56df2336fadf58e98672cdbfbeb913d304733 into 21ed72d35f6b26d3670d1bfba22a...
Committer Matthew Evans
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 396

@ml-evs
Copy link
Member Author

ml-evs commented Oct 18, 2024

Just a note before merging that the original behaviour (but working) should be reintroduced in #943.

@ml-evs ml-evs merged commit 96ac6a9 into main Oct 18, 2024
17 checks passed
@ml-evs ml-evs deleted the ml-evs/quickfix-file-overwriting branch October 18, 2024 10:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working priority/high
Projects
None yet
Development

Successfully merging this pull request may close these issues.

stored_files name clash leads to leakage between samples
2 participants