{"action":"created","issue":{"url":"https://api.github.com/repos/DjangoGirls/djangogirls/issues/13","labels_url":"https://api.github.com/repos/DjangoGirls/djangogirls/issues/13/labels{/name}","comments_url":"https://api.github.com/repos/DjangoGirls/djangogirls/issues/13/comments","events_url":"https://api.github.com/repos/DjangoGirls/djangogirls/issues/13/events","html_url":"https://github.com/DjangoGirls/djangogirls/issues/13","id":53930283,"number":13,"title":"catch-all url for event","user":{"login":"rebkwok","id":6770950,"avatar_url":"https://avatars.githubusercontent.com/u/6770950?v=3","gravatar_id":"","url":"https://api.github.com/users/rebkwok","html_url":"https://github.com/rebkwok","followers_url":"https://api.github.com/users/rebkwok/followers","following_url":"https://api.github.com/users/rebkwok/following{/other_user}","gists_url":"https://api.github.com/users/rebkwok/gists{/gist_id}","starred_url":"https://api.github.com/users/rebkwok/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rebkwok/subscriptions","organizations_url":"https://api.github.com/users/rebkwok/orgs","repos_url":"https://api.github.com/users/rebkwok/repos","events_url":"https://api.github.com/users/rebkwok/events{/privacy}","received_events_url":"https://api.github.com/users/rebkwok/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-01-09T23:42:14Z","updated_at":"2015-01-10T12:42:58Z","closed_at":null,"body":"Hi\r\n\r\nIn core/urls.py, there is this line:\r\nurl(r'^(?P<city>[\\w\\d/]+)$', 'core.views.event', name='event'),\r\n\r\nThis basically says \"match djangogirls.org/ plus any combination of letters and go to the event view\".  Then in core/views.py, there is a try/except clause on line 50 that says if the event page doesn't exist, redirect to /events.  When developing the jobs site, we've found that the url for /jobs/ redirects to /events when the trailing slash is omitted - the event url plus the redirect is a catchall url, so we can never get a 404 (any nonsense url e.g. djangogirls.org/sdfshdjkfkasejaoefjasdlk redirects to djangogirls.org/events).  \r\n\r\nI can see that a workaround redirect has already been done for /admin in djangogirls/urls.py, but if the site is being expanded with more apps/sections, this will mean that it will need to be done for every url.  Is this how you'd prefer it to be done, or would it be better to add a prefix for event (/event/berlin etc - which admittedly will need to deal with the event pages already set up)\r\n\r\nThanks\r\nBecky\r\n"},"comment":{"url":"https://api.github.com/repos/DjangoGirls/djangogirls/issues/comments/69454527","html_url":"https://github.com/DjangoGirls/djangogirls/issues/13#issuecomment-69454527","issue_url":"https://api.github.com/repos/DjangoGirls/djangogirls/issues/13","id":69454527,"user":{"login":"olasitarska","id":527795,"avatar_url":"https://avatars.githubusercontent.com/u/527795?v=3","gravatar_id":"","url":"https://api.github.com/users/olasitarska","html_url":"https://github.com/olasitarska","followers_url":"https://api.github.com/users/olasitarska/followers","following_url":"https://api.github.com/users/olasitarska/following{/other_user}","gists_url":"https://api.github.com/users/olasitarska/gists{/gist_id}","starred_url":"https://api.github.com/users/olasitarska/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/olasitarska/subscriptions","organizations_url":"https://api.github.com/users/olasitarska/orgs","repos_url":"https://api.github.com/users/olasitarska/repos","events_url":"https://api.github.com/users/olasitarska/events{/privacy}","received_events_url":"https://api.github.com/users/olasitarska/received_events","type":"User","site_admin":false},"created_at":"2015-01-10T12:42:58Z","updated_at":"2015-01-10T12:42:58Z","body":"Hi @rebkwok!\r\n\r\nYeah, I am aware of this for a while. I'd rather fix this (so all urls have / at the end, for example), than to change the url of every event we have. Especially if we're printing posters & stuff, djangogirls.org/london is better and easier to remember than djangogirls.org/event/london.\r\n\r\ndjangogirls.org/london/ is not as bad as djangogirls.org/event/london. We should redirect everything that doesn't have the last / to the same address with /. \r\n\r\nIf the event does not exist, we could show 404 (but maybe with a big link to the events page?)\r\n\r\nDo you want to work on this?\r\n\r\nThanks,\r\nOla"}}