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

Warning on Upload page #8

Open
laceysanderson opened this issue Apr 28, 2017 · 5 comments
Open

Warning on Upload page #8

laceysanderson opened this issue Apr 28, 2017 · 5 comments
Assignees

Comments

@laceysanderson
Copy link
Member

There has been some confusion between the upload and backup pages since they look so similar. As such, it might be prudent to add a warning to the upload page indicating that this should only be done once per dataset once data collection has been completed. It would be helpful to point them to the backups page.

@carolyncaron
Copy link
Member

carolyncaron commented Sep 22, 2017

In addition to this, we are concerned that users may still attempt to upload the same file more than once without realizing that all versions would then be added to the database. Perhaps the system could detect additional upload attempts from the same account within a specified timeframe, and provide a warning that if they made corrections to a file that has already been uploaded, to please contact us rather than uploading again.

Our real-life example of this is one collaborator who initiated an upload, made it to stage 2 but realized they were missing a new trait in their file that they wished to define. We suspect they hit the back button to return to step 1, uploaded their newly edited file and continued to the last step. Fortunately, we have just one copy of the data stored in the database but there was nothing to prevent them from reaching the final step and THEN returning to upload their additional column (with all previously uploaded columns still filled in!!!) and this would have created a nightmare to resolve if we didn't catch it right away.

This is a tricky problem since we don't want to discourage multiple uploads if the files originated from different phenotypers (who potentially share the same account), or multiple files with blank columns, such that when all files are uploaded, every column is filled in only once (also likely due to multiple phenotypers but not a given).

@laceysanderson
Copy link
Member Author

@reynoldtan was this closed because it's been done? And if so, can you point to a commit or pull request?

@reynoldtan
Copy link
Member

reynoldtan commented Apr 25, 2018

@laceysanderson I don't recall, but I think it was not done due to @carolyncaron comments required more clarification/discussion and might be difficult to implement. Let's reactivate this issue. Please review text for the warning message:

File upload to this page should only be done once per dataset once data collection has been completed. To create a backup copy of your spreadsheet file, please go to Backup Page.

@reynoldtan reynoldtan reopened this Apr 26, 2018
@reynoldtan
Copy link
Member

@carolyncaron and @laceysanderson for preventing a user from uploading file/data more than once.

Suggestions:

  1. Detect back button as in your example as an attempt to upload twice or just having problem uploading thus we warn the user to contact us.
  2. Check the first x number of rows of the file and see if it matched in the system, then we can tell that the same file/data was previously uploaded. Taking into consideration the person uploading, job queue and date.
  3. Stop user from initiating upload job simultaneously.

@laceysanderson
Copy link
Member Author

Hmmm... hold off on this one reynold. Too much on my plate to consider it properly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants