Otevírání článků na správné straně #54
Merged
zelvuska
merged 4 commits from clanek-strana
into master
5 months ago
Loading…
Reference in new issue
There is no content yet.
Delete Branch 'clanek-strana'
Deleting a branch is permanent. It CANNOT be undone. Continue?
Přidal jsem optional sloupeček
strana
do tabulky článků.Ještě je potřeba něco udělat s migrací.
Vypadá to super :)
Mně by se víc líbilo, kdyby to mohlo být i něco jiného než
#page=N
(tj. aby to nebylo číslo, ale string, který potenciálně může obsahovatpage=42
).Já tady budu nesouhlasit s Jidášem. Přijde mi, že když to bude jen číslo strany, tak aspoň běžný org zvládne přidat odkaz na článek správně, naopak když to bude magický (also: netuším, jestli je někde specifikovaný formát toho fragmentu) string, tak to už navždycky budou dělat webaři. A napsat migraci na to, aby se to kdyžtak přepsalo na
page=ČÍSLO
je ten malý problém ve výsledku. Takže preferuji (aspoň dokud články začínají na nové stránce) mít jen číslo stránky.Co se migrace týče, prostě z ní ručně smaž řádky 18…27 (v poli
migrations
zbyde jen jedenAddField
), a asi není potřeba to víc komplikovat… S těmi treenode-haluzemi něco provedu do konce semestru, jen to teď nechci míchat do nesouvisejících věcí…ups, čtu po sobě můj první dnešní komentář a je trochu moc zkratkovitý. Ten kec o migraci (3. věta) se snaží říct, že až někdo bude mít použití pro něco obecnějšího, tak se to dá snadno převést
Odstranil jsem ten kekel z migrace (sorry, to jsme měli opravit dřív, než jsme vás zaučovali).
Také se číslo stránky nevkládalo do toho URL, protože tam bylo
clanek.cislo.strana
místoclanek.strana
. (Příště to chce otestovat to lokálně…)476f8263d4
into master 5 months agoNyní můžeme sehnat nějakého nevebaře-dobrovolníka, který to (třeba za jídlo) projde a přidá tam ty stránky :D
476f8263d4
.