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

Error raised when using more than one / in the checklist id #14

Closed
stephane-dereppe opened this issue Jul 13, 2016 · 1 comment
Closed
Milestone

Comments

@stephane-dereppe
Copy link

stephane-dereppe commented Jul 13, 2016

The checklist app do not work if we set a template id with more than one /

ex : /vas/governance/development/architectureCheck

It is also impossible to delete a checklist template with a name like that..

Possible Quick fix : do not allow template name with more than one /

@kullervo16
Copy link
Owner

allowing multiple / in the path screws up the URL structure that was defined during the REST cleanup, so indeed a large impact...

not being able to delete them only applies to checklists created with versions prior to 0.8. You can delete/move them manually on the file store and then restart the repo.

Fix will be to refuse values with more than 1 / in the name client side..

@kullervo16 kullervo16 added this to the 0.9.2 milestone Jul 24, 2016
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