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

Více orgů k jednomu PDF v korekturovátku #52

Merged
zelvuska merged 4 commits from vice_orgu_v_korekturovatku into master 2024-10-23 12:47:49 +02:00
Conversation 4 Commits 4 Files changed 4 +50 -9
ledoian commented 2024-06-12 02:16:21 +02:00
Owner
Copy link

Tož, featura se implementovala. Lokálně vypadá elementárně funkčně. Má ale několik nedostatků:

  • Nic orgům nebrání teď vybírat orgy úplně náhodně. To je ale malý problém, do teď to taky nebránilo.
  • Implementace má implicitní vazebnou tabulku, s explicitní neumím udělat horizontální widget na výběr orgů, jen pomocí inlinů. (Defaultní <select multiple> nemá ani vyhledávání :-/) autocomplete nemůže fungovat (spadne na systémovém checku)
  • Inliny by možná byly trochu hezčí řešení, ale jsou až úplně dole a to je neintuitivní imho
  • Výběr orgů zabírá hodně místa, takže se mi přidání PDF nevejde na jeden monitor
  • v přehledu PDF v adminu nejsou teď vidět autoři. Dá se doimplementovat property, která by se zobrazila, ale z dokumentace se zdálo, že by se vykonal DB dotaz pro každou jednu položku, což je trochu nechutné: https://docs.djangoproject.com/en/5.0/ref/contrib/admin/#django.contrib.admin.ModelAdmin.list_display

Možná můžeme schovat věci typu datum a počet stran do nějakého skrytého fieldsetu, tím se problém asi trochu omezí. Případně do toho žďuchat ještě dál/jinak, nevím.

Možná se dá prozkoumat, jak se používají generické vazby jako inline, ale moc tomu nedávám… https://docs.djangoproject.com/en/5.0/ref/contrib/admin/#using-generic-relations-as-an-inline

Tož, featura se implementovala. Lokálně vypadá elementárně funkčně. Má ale několik nedostatků: - Nic orgům nebrání teď vybírat orgy úplně náhodně. To je ale malý problém, do teď to taky nebránilo. - Implementace má implicitní vazebnou tabulku, s explicitní ~~neumím udělat horizontální widget na výběr orgů, jen pomocí inlinů. (Defaultní `<select multiple>` nemá ani vyhledávání :-/)~~ autocomplete nemůže fungovat (spadne na systémovém checku) - ~~Inliny by možná byly trochu hezčí řešení, ale jsou až úplně dole a to je neintuitivní imho~~ - ~~Výběr orgů zabírá hodně místa, takže se mi přidání PDF nevejde na jeden monitor~~ - v přehledu PDF v adminu nejsou teď vidět autoři. Dá se doimplementovat property, která by se zobrazila, ale z dokumentace se zdálo, že by se vykonal DB dotaz pro každou jednu položku, což je trochu nechutné: <https://docs.djangoproject.com/en/5.0/ref/contrib/admin/#django.contrib.admin.ModelAdmin.list_display> Možná můžeme schovat věci typu datum a počet stran do nějakého skrytého fieldsetu, tím se problém asi trochu omezí. Případně do toho žďuchat ještě dál/jinak, nevím. Možná se dá prozkoumat, jak se používají generické vazby jako inline, ale moc tomu nedávám… <https://docs.djangoproject.com/en/5.0/ref/contrib/admin/#using-generic-relations-as-an-inline>
ledoian added 1 commit 2024-06-12 02:16:22 +02:00
ledoian commented 2024-06-12 02:24:51 +02:00
Author
Owner
Copy link

Ha, ono to umí i Select2Multiple. Půlka problémů tím padá…

Ha, ono to umí i Select2Multiple. Půlka problémů tím padá…
ledoian added 1 commit 2024-06-12 02:25:28 +02:00
ledoian commented 2024-06-12 02:26:48 +02:00
Author
Owner
Copy link

Jen nevím, jestli autocomplete dodržuje pořadí, tj. jestli nabízí aktivní orgy na začátku. Zrovna tady to asi hodně chceme, TODO otestovat na testwebu. (Teď už to dělat nebudu.)

Jen nevím, jestli autocomplete dodržuje pořadí, tj. jestli nabízí aktivní orgy na začátku. Zrovna tady to asi hodně chceme, TODO otestovat na testwebu. (Teď už to dělat nebudu.)
ledoian added 1 commit 2024-06-15 22:50:35 +02:00
zelvuska added 1 commit 2024-10-23 12:25:21 +02:00
zelvuska commented 2024-10-23 12:43:43 +02:00
Owner
Copy link

Vypadá funkčně, proti kódu nic nemám :)

Vypadá funkčně, proti kódu nic nemám :)
zelvuska commented 2024-10-23 12:47:43 +02:00
Owner
Copy link

Jen nevím, jestli autocomplete dodržuje pořadí, tj. jestli nabízí aktivní orgy na začátku. Zrovna tady to asi hodně chceme, TODO otestovat na testwebu. (Teď už to dělat nebudu.)

Ano dodržuje :)

> Jen nevím, jestli autocomplete dodržuje pořadí, tj. jestli nabízí aktivní orgy na začátku. Zrovna tady to asi hodně chceme, TODO otestovat na testwebu. (Teď už to dělat nebudu.) Ano dodržuje :)
zelvuska merged commit c7910ed72a into master 2024-10-23 12:47:49 +02:00
zelvuska deleted branch vice_orgu_v_korekturovatku 2024-10-23 12:47:50 +02:00
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
2 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#52
No description provided.
Delete branch "vice_orgu_v_korekturovatku"

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?