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

Adding duplicate resources #296

Closed
clash99 opened this issue Jun 30, 2016 · 4 comments
Closed

Adding duplicate resources #296

clash99 opened this issue Jun 30, 2016 · 4 comments

Comments

@clash99
Copy link
Contributor

clash99 commented Jun 30, 2016

It would be great if adding a duplicate resource (by name comparison perhaps?), it would alert you if that resource already existed and ask you if you wanted to connect to it, or continue to upload your new resource (which I would assume at this point is not a duplicate).

@wonderchook
Copy link
Contributor

What about comparing a hash of the file to what is already uploaded in the project? @oliverroick did we talk about this at some point or is my memory bad?

@oliverroick
Copy link
Member

We did talk about updating XLSForms.

Using hashes to look for existing files is something we can definitely do. Do bear in mind, though: We went a long way to making sure files don't have to go through our servers when uploading resources. Instead, upload them to S3 directly and just store its URL. If we want to check if a file exists, we have to download and process the files every time a resource is being created.

@wonderchook wonderchook added this to the Post July Release Sprint milestone Jul 9, 2016
@wonderchook wonderchook modified the milestones: Version 1.0, Post July Release Sprint Jul 21, 2016
@wonderchook
Copy link
Contributor

I think this can wait for the time being.

@dpalomino
Copy link

It seems we are not going to implement this any time in the short-medium term, so closing it for now...

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

6 participants