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

Vylepšení galerie: účastnická privátní galerie #95

Open
ledoian wants to merge 0 commits from galerie_vylepseni into master
pull from: galerie_vylepseni
merge into: mam:master
Conversation 9 Commits - Files changed -
ledoian commented 2025-04-23 21:10:26 +02:00
Owner
Copy link

Klidně rovnou komentujte změny, zatím postupně vzniká (a chci mít možnost to pushovat), ale změny jsou snad vesměs finální.

No, možná se někdy rozhodnu, že to celé refaktoruju, ale to bude separátní commit…

Klidně rovnou komentujte změny, zatím postupně vzniká (a chci mít možnost to pushovat), ale změny jsou snad vesměs finální. No, možná se někdy rozhodnu, že to celé refaktoruju, ale to bude separátní commit…
ledoian added 1 commit 2025-04-23 21:10:27 +02:00
ledoian added 1 commit 2025-04-23 23:03:42 +02:00
ledoian commented 2025-04-23 23:06:40 +02:00
Author
Owner
Copy link

Co se týče účastnických galerií, views by měly být hotové, zbývá dořešit logiku v templatech (kdy komu umožnit přeuspořádat galerie, rámečky kolem neveřejných věcí).

Klidně se mi v tom už šťourejte, a kdyby někdo chtěl napsat testy (že tomu kterému uživateli jenž byl/nebyl na sousu je správně hozeno 404), tak se klidně čiňte :-D

Co se týče účastnických galerií, views by měly být hotové, zbývá dořešit logiku v templatech (kdy komu umožnit přeuspořádat galerie, rámečky kolem neveřejných věcí). Klidně se mi v tom už šťourejte, a kdyby někdo chtěl napsat testy (že tomu kterému uživateli jenž byl/nebyl na sousu je správně hozeno 404), tak se klidně čiňte :-D
ledoian added 2 commits 2025-04-24 00:40:27 +02:00
ledoian commented 2025-04-24 00:52:46 +02:00
Author
Owner
Copy link

Aktuálně by mělo být funkční a možno nasadit. Zbývá:

  • v přehledu galerií barevně odlišit, kdo vidí které galerie (viz vlákno na Matrixu, 2025-04-23 24:25)
  • otestovat na testwebu na reálných datech

Nebylo by k zahození:

  • napsat test, že správní uživatelé dostávají správně 404
  • refaktorovat zbytek kódu, konkrétně:
    • dořešit pár nových FIXME
    • předělat výčet v modelech na Enum a importovat použitelně
    • použít v templatech {% url … %} místo relativních adres
    • dodělat odkaz z Admina na příslušnou galerii (a ideálně i opačně z galerie přes fialový pruh zpět do admina)
    • přejmenovat views (aktuálně se jmenují nahled a detail)
    • asi přejmenovat templates a všechno, aby se nejmenovalo „náhled“ a „galerie“
    • pochopit a pak definitivně smazat ten zakomentovaný kód v galerie/models.py, který nikdy ani nebyl odkomentovaný…

Videjko nebude teď, založím si separátní PR (nejspíš bude dost ortogonální…)

Aktuálně by mělo být funkční a možno nasadit. Zbývá: - [x] v přehledu galerií barevně odlišit, kdo vidí které galerie (viz vlákno na Matrixu, 2025-04-23 24:25) - [x] otestovat na testwebu na reálných datech Nebylo by k zahození: - [ ] napsat test, že správní uživatelé dostávají správně 404 - [ ] refaktorovat zbytek kódu, konkrétně: - [ ] dořešit pár nových FIXME - [x] předělat výčet v modelech na `Enum` a importovat použitelně - [ ] použít v templatech `{% url … %}` místo relativních adres - [ ] dodělat odkaz z Admina na příslušnou galerii (a ideálně i opačně z galerie přes fialový pruh zpět do admina) - [ ] přejmenovat views (aktuálně se jmenují `nahled` a `detail`) - [ ] asi přejmenovat templates a všechno, aby se nejmenovalo „náhled“ a „galerie“ - [x] pochopit a pak definitivně smazat ten zakomentovaný kód v `galerie/models.py`, který nikdy ani nebyl odkomentovaný… Videjko nebude teď, založím si separátní PR (nejspíš bude dost ortogonální…)
ledoian changed title from WIP: Vylepšení galerie: účastnická privátní galerie, videjka to Vylepšení galerie: účastnická privátní galerie 2025-04-24 00:52:55 +02:00
ledoian reviewed 2025-04-24 01:09:40 +02:00
galerie/views.py Outdated
@ -33,2 +51,4 @@
galerie = get_object_or_404(Galerie, pk=pk)
soustredeni = top_galerie(galerie).soustredeni
# FIXME: přepsat model a použít přímo dolů…
ledoian commented 2025-04-24 01:09:29 +02:00
Author
Owner
Copy link

Než zapomenu, co se tím míní: u Obrázků je pattern galerie.obrazek_set.order_by(…), u galerií by se mi líbil taky (tj. nadgalerie.podgalerie_set.…, nebo to aspoň udělat konzistentně). Field galerie_up ale nemá related_name, takže se příslušný pseudo-field jmenuje galerie_set – bylo by dobré ho mít pojmenovaný explicitně, třeba jako podgalerie – a pak celý view přepsat konzistentně (někde níž se dokonce používá skvělá konstrukce galerie.galerie_up.galerie_set.all() :-D)

Než zapomenu, co se tím míní: u Obrázků je pattern `galerie.obrazek_set.order_by(…)`, u galerií by se mi líbil taky (tj. `nadgalerie.podgalerie_set.…`, nebo to aspoň udělat konzistentně). Field `galerie_up` ale nemá `related_name`, takže se příslušný pseudo-field jmenuje `galerie_set` – bylo by dobré ho mít pojmenovaný explicitně, třeba jako `podgalerie` – a pak celý view přepsat konzistentně (někde níž se dokonce používá skvělá konstrukce `galerie.galerie_up.galerie_set.all()` :-D)
ledoian added 1 commit 2025-04-24 01:29:43 +02:00
ledoian added 1 commit 2025-04-24 01:32:14 +02:00
zelvuska added 1 commit 2025-04-30 20:06:30 +02:00
zelvuska added 1 commit 2025-04-30 20:09:22 +02:00
zelvuska added 1 commit 2025-04-30 20:11:03 +02:00
zelvuska added 1 commit 2025-04-30 20:31:22 +02:00
zelvuska added 1 commit 2025-04-30 20:56:04 +02:00
zelvuska added 1 commit 2025-04-30 21:00:35 +02:00
zelvuska added 1 commit 2025-04-30 21:02:52 +02:00
zelvuska commented 2025-04-30 21:06:04 +02:00
Owner
Copy link

Lepší než 404 by bylo vracet 403…

Lepší než 404 by bylo vracet 403…
zelvuska commented 2025-04-30 21:21:36 +02:00
Owner
Copy link

Zvednout limit u galerií a odkomentovat odkaz na originál.

Zvednout limit u galerií a odkomentovat odkaz na originál.
zelvuska approved these changes 2025-04-30 21:25:31 +02:00
zelvuska commented 2025-04-30 21:30:32 +02:00
Owner
Copy link

Zlikvidovat „NIKDO“.

Zlikvidovat „NIKDO“.
ledoian commented 2025-05-05 02:17:30 +02:00
Author
Owner
Copy link

Mám bug: v galerie/templates/galerie/GalerieNahled.html na konci je

  {% if galerie.zobrazit > 0 %}
  {% if galerie.zobrazit == 1 or galerie.zobrazit == 2 %}
    </div> {# mam-org-only #}
  {% elif galerie.zobrazit == 2 %}
    </div> {# mam-resitel-only #}
  {% endif %}

v elifu má být == 3 (a nebo nesedí komentáře)…

Mám bug: v `galerie/templates/galerie/GalerieNahled.html` na konci je ``` {% if galerie.zobrazit > 0 %} {% if galerie.zobrazit == 1 or galerie.zobrazit == 2 %} </div> {# mam-org-only #} {% elif galerie.zobrazit == 2 %} </div> {# mam-resitel-only #} {% endif %} ``` v `elif`u má být `== 3` (a nebo nesedí komentáře)…
ledoian commented 2025-05-05 17:23:12 +02:00
Author
Owner
Copy link

Přemýšlím, jestli nebude lepší založit na refactoring nový PR, dát tam místní TODO jako úvodní post a tenhle zavřít, čistě proto, že tady se ty poznámky blbě dohledají a nedává smysl sem připisovat další…

Přemýšlím, jestli nebude lepší založit na refactoring nový PR, dát tam místní TODO jako úvodní post a tenhle zavřít, čistě proto, že tady se ty poznámky blbě dohledají a nedává smysl sem připisovat další…
ledoian commented 2025-12-02 19:01:00 +01:00
Author
Owner
Copy link

v elifu má být == 3 (a nebo nesedí komentáře)…

Už tam je (zatím u mně, až to pushnu, tak bude i na gitei)

> v `elif`u má být `== 3` (a nebo nesedí komentáře)… Už tam je (zatím u mně, až to pushnu, tak bude i na gitei)
ledoian commented 2025-12-02 19:12:07 +01:00
Author
Owner
Copy link

Lepší než 404 by bylo vracet 403…

Záleží, jestli chceme spíš popřít existenci dalších galerií, nebo ji přiznat.

A kromě toho nevím, jestli 403 umí dávat další vysvětlení toho, co se stalo – django.http.Http404 jde zavolat s parametrem, který pak jde vidět v debug módu… (Mám velmi matné tušení, že PermissionDenied to neumělo, ale je to spíš matné tušení než že bych mělo důkaz…)

  • Ale nechám si to tu jako úkol a třeba to jednou dohledám :-D
> Lepší než 404 by bylo vracet 403… Záleží, jestli chceme spíš popřít existenci dalších galerií, nebo ji přiznat. A kromě toho nevím, jestli 403 umí dávat další vysvětlení toho, co se stalo – `django.http.Http404` jde zavolat s parametrem, který pak jde vidět v debug módu… (Mám velmi matné tušení, že `PermissionDenied` to neumělo, ale je to spíš matné tušení než že bych mělo důkaz…) - [ ] Ale nechám si to tu jako úkol a třeba to jednou dohledám :-D
This branch is already included in the target branch. There is nothing to merge.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin galerie_vylepseni:galerie_vylepseni
git checkout galerie_vylepseni

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git checkout master
git merge --no-ff galerie_vylepseni
git checkout galerie_vylepseni
git rebase master
git checkout master
git merge --ff-only galerie_vylepseni
git checkout galerie_vylepseni
git rebase master
git checkout master
git merge --no-ff galerie_vylepseni
git checkout master
git merge --squash galerie_vylepseni
git checkout master
git merge --ff-only galerie_vylepseni
git checkout master
git merge galerie_vylepseni
git push origin master
Sign in to join this conversation.
Reviewers
No reviewers
zelvuska
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#95
No description provided.
Delete branch "galerie_vylepseni"

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?