Otevírání článků na správné straně #54

Merged
zelvuska merged 4 commits from clanek-strana into master 3 months ago
koskja commented 3 months ago
Collaborator

Přidal jsem optional sloupeček strana do tabulky článků.
Ještě je potřeba něco udělat s migrací.

Přidal jsem optional sloupeček `strana` do tabulky článků. Ještě je potřeba něco udělat s migrací.
koskja added 1 commit 3 months ago
3c3b9f755a Add `strana` column to `Clanek`
koskja added 1 commit 3 months ago
Owner

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 obsahovat page=42).

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 obsahovat `page=42`).
Owner

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.

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.
Owner

Co se migrace týče, prostě z ní ručně smaž řádky 18…27 (v poli migrations zbyde jen jeden AddField), 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í…

Co se migrace týče, prostě z ní ručně smaž řádky 18…27 (v poli `migrations` zbyde jen jeden `AddField`), 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í…
Owner

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

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
zelvuska added 1 commit 3 months ago
zelvuska added 1 commit 3 months ago
Owner

Odstranil jsem ten kekel z migrace (sorry, to jsme měli opravit dřív, než jsme vás zaučovali).

Odstranil jsem ten kekel z migrace (sorry, to jsme měli opravit dřív, než jsme vás zaučovali).
Owner

Také se číslo stránky nevkládalo do toho URL, protože tam bylo clanek.cislo.strana místo clanek.strana. (Příště to chce otestovat to lokálně…)

Také se číslo stránky nevkládalo do toho URL, protože tam bylo `clanek.cislo.strana` místo `clanek.strana`. (Příště to chce otestovat to lokálně…)
zelvuska merged commit 476f8263d4 into master 3 months ago
zelvuska deleted branch clanek-strana 3 months ago
Owner

Nyní 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

Nyní 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
The pull request has been merged as 476f8263d4.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.