oSoc14 / code9000

Calendar app for schools
MIT License
1 stars 4 forks source link

Use IDs in export URLs #51

Closed Dobbelsteen closed 9 years ago

Dobbelsteen commented 9 years ago

Right now we are using schoolnames and group names in the URL so that it's clear for users what they're getting. This however requires a lot of hard coded functionality in various controllers to work correctly. I suggest we simply use IDs as the first parameters where necessery, and add the school and group names at the end as trash parameters which will be ignored by the app. This allows us to cut out a lot of ugly code that can potentially cause a lot of bugs (duplicate school names etc are impossible)

Now:

http://localhost:82/OKFN/public/export/schoolname/groupname

Suggested:

http://localhost:82/OKFN/public/export/{schoolid}/{groupid}/schoolname/groupname