Put current conference in url #14

2015-06-09T21:54:38+03:00

AndrewRadev commented

2015-06-09 21:54:38 +03:00

(Migrated from github.com)

Идеята е да имаме url-и като /management/OpenFest-2015/events. Параметъра за scope ще се set-ва с default_url_params, и така няма да трябва да го предаваме навсякъде.

Един недостатък е, че url-ите, които не са scope-нати под ":current_conference" ще им се добави отзад ?current_conference=OpenFest-2015, което ще изглежда малко странно. Не бих казал, че е голям проблем.

Следващата стъпка (ако решим да правим това) ще е да се сложи slug поле в базата, което може да се генерира от английския превод на title-а, макар че може би ще е добре да го оставим editable поле. Може и да ползваме нещо като conference-#{id}, ако само и единствено ни пука за linkability и не толкова за това колко смислени изглеждат url-ите, ама ако ще са linkable, поне нека да са адекватно четими.

Идеята е да имаме url-и като `/management/OpenFest-2015/events`. Параметъра за scope ще се set-ва с `default_url_params`, и така няма да трябва да го предаваме навсякъде. Един недостатък е, че url-ите, които не са scope-нати под ":current_conference" ще им се добави отзад `?current_conference=OpenFest-2015`, което ще изглежда малко странно. Не бих казал, че е голям проблем. Следващата стъпка (ако решим да правим това) ще е да се сложи slug поле в базата, което може да се генерира от английския превод на title-а, макар че може би ще е добре да го оставим editable поле. Може и да ползваме нещо като `conference-#{id}`, ако само и единствено ни пука за linkability и не толкова за това колко смислени изглеждат url-ите, ама ако ще са linkable, поне нека да са адекватно четими.

ignisf commented

2015-07-12 11:56:26 +03:00

(Migrated from github.com)

👍

:+1:

Sign in to join this conversation.

No reviewers