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

invalid URLs to download forms #126

Open
danbjoseph opened this issue Jun 5, 2016 · 2 comments
Open

invalid URLs to download forms #126

danbjoseph opened this issue Jun 5, 2016 · 2 comments

Comments

@danbjoseph
Copy link
Member

form_id versus the file name seems to be causing issues. the link to download a form through the main interface appears to be using the form_id to build the download url, whereas the filename is what needs to be used (i.e. in the screen shots for the example shown below it should be http://omkserver.com/omk/data/forms/africa_schools.xlsx and not http://omkserver.com/omk/data/forms/schools.xlsx). not sure what other implications this might have in terms of form listing when downloading, preventing accidental overwrites, etc. i've seen plenty of times in the past when someone will update a filename for a new version of a form but will leave the meta-data defined within the form the same.

  • the main interface

screen shot 2016-06-05 at 7 39 00 am

- the url when you click the download the xls

screen shot 2016-06-05 at 7 38 52 am

- the form file

screen shot 2016-06-05 at 7 38 23 am

screen shot 2016-06-05 at 7 48 38 am

@hallahan
Copy link
Contributor

The issue isn't really a form name / form_id discrepancy. Actually, it's a file name / form_id discrepancy.

When an XLSX file is uploaded, we should take a look at the actual form ID and write the file as that. The user should be notified of that rewrite. For now, make sure you name your xlsx file the same thing as your form_id.

@hallahan
Copy link
Contributor

I'd say we really want a tiny little node module that checks these things:

  • Is the form id already on the server?
  • Is the file name the same?
  • Notify the user about this and what we did about it?
  • Ask if we want to overwrite the existing form or make a new one with a number at the end?

This issue is in the wrong repo: should be https://github.com/AmericanRedCross/OpenMapKitServer/issues

Solving this would resolve #53

@danbjoseph danbjoseph transferred this issue from posm/posm Aug 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants