-
Notifications
You must be signed in to change notification settings - Fork 541
4813 allow duplicate files #6924
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
Merged
Merged
Changes from 73 commits
Commits
Show all changes
83 commits
Select commit
Hold shift + click to select a range
f311312
#4813 allow duplicate files to be uploaded
sekmiller cfe629d
Merge branch 'develop' into 4813-allow-duplicate-files
sekmiller abcaa3b
#4813 Allow replacement with the same file
sekmiller ce34432
Merge branch 'develop' into 4813-allow-duplicate-files
sekmiller 4743b23
#4813 update comments
sekmiller ca7ba78
#4813 update test and bundle
sekmiller 17869df
#4813 fix failing test
sekmiller 34ce725
Merge branch 'develop' into 4813-allow-duplicate-files
sekmiller 9b4152c
#4813 fix filemetadata testing
sekmiller 801d4bb
Create 4813-allow-duplicate-files.md
djbrooke d1139a9
Merge branch 'develop' into 4813-allow-duplicate-files
sekmiller aa02d34
#4813 remove extraneous logging
sekmiller 0cea0da
#4813 fix display of file names in popup
sekmiller 2997792
Merge branch 'develop' into 4813-allow-duplicate-files
sekmiller 121620c
#4813 Do not allow replace with same file
sekmiller 5012eb6
Merge branch 'develop' into 4813-allow-duplicate-files
sekmiller b24b3c2
#4813 modify inline error message for dupes
sekmiller 1182c54
#4813 add more detail to duplicate messages
sekmiller 6fb7c93
#4813 grammar
sekmiller 86294eb
Merge branch 'develop' into 4813-allow-duplicate-files
sekmiller 21a2a7f
#4813 update button label
sekmiller f82ca33
Merge branch 'develop' into 4813-allow-duplicate-files
sekmiller a323db5
#4813 add note about duplicate file path/name
sekmiller 178b6a4
#4813 add doc for duplicate file content/name
sekmiller 640325c
#4813 add link to user guide to popup
sekmiller eec2854
#4813 reword inline warning message
sekmiller 0273d86
Merge branch 'develop' into 4813-allow-duplicate-files
sekmiller fae5dc1
#4813 add warning if replacement file is dup of existing file
sekmiller d36c046
#4813 show replace with dup message on upload component
sekmiller 847fd62
updating rules, need to check with @sekmiller
djbrooke 6eddfc3
Merge branch 'develop' into 4813-allow-duplicate-files
sekmiller 954b3b7
update release notes for dupe file handling
djbrooke 0328487
update duplicate file explanation as it pertains to replace
djbrooke 0e90ce8
typo, thanks @jggautier
djbrooke 911cd0e
#4813 update replace rules
sekmiller c8f358f
#4813 update bundle to reflect new section in User Guide
sekmiller d8dcb17
Merge branch 'develop' into 4813-allow-duplicate-files
sekmiller a8629f4
Merge branch 'develop' into 4813-allow-duplicate-files
sekmiller 2d79816
Merge branch 'develop' into 4813-allow-duplicate-files
sekmiller 70820ba
Merge branch 'develop' into 4813-allow-duplicate-files
sekmiller dcf2d73
adding a
djbrooke 1b68d11
Merge branch 'develop' into 4813-allow-duplicate-files
sekmiller f1f18a0
#4813 update formatting of inline message
sekmiller e9f24cf
Merge branch 'develop' into 4813-allow-duplicate-files
sekmiller dfe9af5
#4813 add render logic to inline warning
sekmiller 6af0f3e
#4813 get update button to display after upload
sekmiller 117b321
Merge branch 'develop' into 4813-allow-duplicate-files
sekmiller 77ad2e3
#4813 update popup messages
sekmiller dab96ab
Merge branch 'develop' into 4813-allow-duplicate-files
sekmiller 6e7c3b9
#4813 add warnings when dup content added in same upload session
sekmiller ea85d9f
Merge branch 'develop' into 4813-allow-duplicate-files
sekmiller 1ebd566
Cleaned up upload/edit file table layout to fix padding and margin is…
mheppler f64d4c6
#4813 change replace with dup to error. no popup.
sekmiller 40701b9
Correct guides typo, fix guides link, ui clean up for duplidate files…
mheppler 9fe91e1
#4813 fix file replace dupe messaging
sekmiller 6eb7b52
Merge branch 'develop' into 4813-allow-duplicate-files
sekmiller c69ef84
#4813 remove unused code
sekmiller 9350502
#4813 fix button refresh on file delete
sekmiller 54011d7
#4813 fix failing test
sekmiller d0f5adc
Merge branch 'develop' into 4813-allow-duplicate-files
sekmiller 36a0d76
#4813 clean up some message code
sekmiller e9d58aa
Merge branch 'develop' into 4813-allow-duplicate-files
sekmiller 77edbb6
Merge branch 'develop' into 4813-allow-duplicate-files
sekmiller 14d24f5
#4813 fix popup header
sekmiller 60f2750
#4813 include new files in dupe name search
sekmiller 8a3ed9b
Merge branch 'develop' into 4813-allow-duplicate-files
sekmiller 21f98be
#4813 fix replace with multiple matching files
sekmiller 0e531ee
Merge branch 'develop' into 4813-allow-duplicate-files
sekmiller 7126aaa
Merge branch 'develop' into 4813-allow-duplicate-files
sekmiller 5522e5d
#4813 add note about multiples to inline message
sekmiller 2d60309
#4813 update messaging
sekmiller f87c9d9
Merge branch 'develop' into 4813-allow-duplicate-files
sekmiller 0f41e9d
#4813 more message updates
sekmiller 8169a1b
Merge branch 'develop' into 4813-allow-duplicate-files
sekmiller cef57dc
#4813 redo update buttons
sekmiller 031b253
#4813 remove file names from delete success msg
sekmiller a4be5eb
Merge branch 'develop' into 4813-allow-duplicate-files
sekmiller 46badd3
#4813 code cleanup
sekmiller 90edc9e
Merge branch 'develop' into 4813-allow-duplicate-files
sekmiller 0e99e15
#4813 clarify file name/path editing rules
sekmiller d0bfd3d
#4813 removing out of date comments
sekmiller 82f9825
#4813 remove marked as dup from file metadata
sekmiller 57ab613
#4813 separate retrieval of files for Deletion
sekmiller File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| We should note that duplicate files are now allowed, and installations may want to contact people now that this is available. Point to rules in the Guides. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
is it worth being more explicit here about upload vs edit? i.e. related to the next bullet about changing the directory, is also changing the name after upload. (so the suggestion is to add that there, and change add to upload here)
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.
I made a change to the doc here, please see that it helps to add some clarity.