-
Notifications
You must be signed in to change notification settings - Fork 18
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
PE-6113: (WIP): add a modal to filter success uploads #1736
base: dev
Are you sure you want to change the base?
Conversation
enum UploadActions { skip, replace } | ||
/// | ||
/// `SkipSuccessfullyUploads` will skip the files that were successfully uploaded. | ||
enum UploadActions { skip, skipSuccessfullyUploads, replace } |
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.
prefer to rename to skipSuccessfulUploads or skipExistingFiles
@@ -3,4 +3,6 @@ | |||
/// `Skip` Will ignore the files and don't upload them. | |||
/// | |||
/// `Replace` will upload the conflicting file and replace the existent. | |||
enum UploadActions { skip, replace } | |||
/// | |||
/// `SkipSuccessfullyUploads` will skip the files that were successfully uploaded. |
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.
nit(grammar):
/// `SkipSuccessfullyUploads` will skip the files that were successfully uploaded. | |
/// `SkipSuccessfulUploads` will skip the files that were successfully uploaded. |
Visit the preview URL for this PR (updated for commit 3a74153): https://ardrive-web--pr1736-pe-6113-fix-d0tmwm13.web.app (expires Fri, 07 Jun 2024 20:17:36 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: a224ebaee2f0939e7665e7630e7d3d6cd7d0f8b0 |
@@ -333,7 +333,12 @@ void main() { | |||
const TypeMatcher<UploadPreparationInProgress>(), | |||
UploadFileConflict( | |||
areAllFilesConflicting: true, | |||
conflictingFileNames: const ['${tRootFolderId}1']), | |||
conflictingFileNames: const [ | |||
'${tRootFolderId}1' |
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.
Curious, why is there a 1 added here?
--- Releases ---
Android release: https://appdistribution.firebase.google.com/testerapps/1:305132849030:android:6cf0cd5ec064fad3ffce07/releases/3hv9t5ai5r108