pyvec / naucse

A server for open course material
Other
2 stars 9 forks source link

API: Remove slashes from course slugs #12

Closed encukou closed 4 years ago

encukou commented 5 years ago

Course "slugs" like courses/mi-pyt and 2019/brno-jaro-knihovny are annoying to handle, and shouldn't be given to API consumers. (In naucse they're a cheap hack for keeping old URLs working).

Naucse should replace the / by -.

encukou commented 4 years ago

I've given this more thought, and I decided that I want to keep identifiers with slashes. If I want to preview pull requests under URLs like gh-pr/<user>/<repo>/<number>, I'll need the slash.

Let me know if this needs more discussion; I'll be happy to reopen.