-
Notifications
You must be signed in to change notification settings - Fork 3
/s issues #56
Comments
Have you changed your mind: #40 (comment)? If we change this, we should change all usages of short URLs to maintain consistency. |
No, I haven't changed my mind, and it's still consistent if we remove it. Short URLs are for things that need to be quickly remembered or conveyed, for example, consider my personal homepage URL: http://inside.mines.edu/~jrosenth The tilde notation is certainly not long, but it is easy to remember the path to my homepage and easy to type it as well. Survey URLs are never "remembered" nor typed manually, therefore they do not need the short URLs. |
Ok. I'm fine with changing to |
created a branch with fixes called survey-Bugs |
@jackrosenthal @sumnerevans I just noticed this enter |
If it's |
::
|
/s
without a survey id fails with 500 (should be 404)/s
with something that cannot be an integer fails (e.g.,/s/foo
)... should be 404/attend
anyway. Thoughts on changing this to/survey
?The text was updated successfully, but these errors were encountered: