mam/mamweb
11
0
Fork 0
Code Issues Pull requests 8 Projects Wiki Activity Actions

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

Merged
zelvuska merged 4 commits from clanek-strana into master 2024-06-20 14:42:40 +02:00
Conversation 7 Commits 4 Files changed 3 +21 -1
koskja commented 2024-06-19 20:34:43 +02:00
Owner
Copy link

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 2024-06-19 20:34:44 +02:00 
Optionally add page fragment to `resitelske_clanky.html`
koskja added 1 commit 2024-06-19 20:34:58 +02:00
zelvuska commented 2024-06-19 20:46:20 +02:00
Owner
Copy link

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`).
ledoian commented 2024-06-19 22:41:58 +02:00
Owner
Copy link

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.
ledoian commented 2024-06-19 22:44:44 +02:00
Owner
Copy link

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í…
ledoian commented 2024-06-19 22:58:54 +02:00
Owner
Copy link

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 2024-06-20 14:32:25 +02:00
zelvuska added 1 commit 2024-06-20 14:38:35 +02:00
zelvuska commented 2024-06-20 14:42:24 +02:00
Owner
Copy link

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).
zelvuska commented 2024-06-20 14:42:33 +02:00
Owner
Copy link

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 2024-06-20 14:42:40 +02:00
zelvuska deleted branch clanek-strana 2024-06-20 14:42:40 +02:00
zelvuska commented 2024-06-20 14:43:41 +02:00
Owner
Copy link

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
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: mam/mamweb#54
No description provided.
Delete branch "clanek-strana"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?