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

upravy_exportu #91

Open
ticvac wants to merge 5 commits from upravy_exportu into master
pull from: upravy_exportu
merge into: mam:master
Conversation 0 Commits 5 Files changed 3 +64 -12
ticvac commented 2025-03-05 18:45:15 +01:00
Owner
Copy link

úpravy podle Tomovych žádosti...

úpravy podle Tomovych žádosti...
ticvac added 2 commits 2025-03-05 18:45:15 +01:00
ticvac added 1 commit 2025-03-05 19:14:14 +01:00
zelvuska reviewed 2025-03-05 19:16:19 +01:00
soustredeni/templates/soustredeni/seznam_soustredeni.html
@ -57,6 +57,7 @@
<a href="../{{soustredeni.pk}}/seznam_ucastniku">HTML tabulka pro tisk</a>,
<a href="../{{soustredeni.pk}}/export_ucastniku">CSV</a>,
zelvuska commented 2025-03-05 19:16:19 +01:00
Owner
Copy link

Nevím, jestli tohle má cenu nechávat, když je tu odkaz na obecné exporty…

Nevím, jestli tohle má cenu nechávat, když je tu odkaz na obecné exporty…
ledoian commented 2025-03-05 20:41:32 +01:00
Owner
Copy link

Vlastně mi přijde, že je tam klidně NějakouDobu™ můžeme nechat – pokud někomu fungovaly / vyhovovaly staré exporty lépe (nevím přesně proč a jak), tak ať je klidně ještě používá… Ale hodilo by se umět nějak zjistit, jak moc to orgové chtějí používat a jaký názor mají na nové exporty…

A možná je tu můžeme <del>škrtnout</del>, aby bylo vidět, že dole jsou Nové™ a Lepší™ exporty?

Vlastně mi přijde, že je tam klidně NějakouDobu™ můžeme nechat – pokud někomu fungovaly / vyhovovaly staré exporty lépe (nevím přesně proč a jak), tak ať je klidně ještě používá… Ale hodilo by se umět nějak zjistit, jak moc to orgové chtějí používat a jaký názor mají na nové exporty… A možná je tu můžeme `<del>`škrtnout`</del>`, aby bylo vidět, že dole jsou Nové™ a Lepší™ exporty?
zelvuska reviewed 2025-03-05 19:17:30 +01:00
personalni/templates/personalni/profil/export_lidi.html
@ -5,12 +5,24 @@
<h2><strong>Export lidí</strong></h2>
<p>Vyberte pole, které chcete exportovat</p>
zelvuska commented 2025-03-05 19:17:30 +01:00
Owner
Copy link

Takže pokud přibyde na řešitelovi další věc, tak se bude muset přidávat sem další položka?

Takže pokud přibyde na řešitelovi další věc, tak se bude muset přidávat sem další položka?
ticvac commented 2025-03-05 19:21:57 +01:00
Author
Owner
Copy link

Ano

Ano
zelvuska approved these changes 2025-03-05 19:28:01 +01:00
soustredeni/templates/soustredeni/seznam_soustredeni.html
@ -57,6 +57,7 @@
<a href="../{{soustredeni.pk}}/seznam_ucastniku">HTML tabulka pro tisk</a>,
<a href="../{{soustredeni.pk}}/export_ucastniku">CSV</a>,
<a href="../{{soustredeni.pk}}/maily_ucastniku">E-maily</a><br>
Exporty pro soustředění: <a href="{% url 'exporty_lidi' %}">exporty</a><br>
zelvuska commented 2025-03-05 19:21:44 +01:00
Owner
Copy link

Ale ty jsou nejenom pro soustředění…

Ale ty jsou nejenom pro soustředění…
ledoian commented 2025-03-05 20:51:05 +01:00
Owner
Copy link

To je podle mě asi jedno, spíš mi chybí, exporty čeho to jsou, případně v jakém formátu…

To je podle mě asi jedno, spíš mi chybí, exporty *čeho* to jsou, případně *v jakém formátu*…
ticvac added 2 commits 2025-03-05 19:49:52 +01:00
zelvuska reviewed 2025-03-05 19:56:40 +01:00
personalni/views.py
@ -167,3 +171,4 @@
data = [{"id": s.id, "display": str(s)} for s in Soustredeni.objects.all()]
return HttpResponse(json.dumps(data), content_type='application/json')
def getFieldsForExport(request):
zelvuska commented 2025-03-05 19:56:35 +01:00
Owner
Copy link

Takže když nastala chyba, tak to udělá něco náhodného?

Takže když nastala chyba, tak to udělá něco náhodného?
ticvac commented 2025-03-05 19:58:29 +01:00
Author
Owner
Copy link

Jako... ona nenastala chyba, uživatel si zažádal o prázdné info... dostane to, co je nastaveno defaultně

Jako... ona nenastala chyba, uživatel si zažádal o prázdné info... dostane to, co je nastaveno defaultně
zelvuska commented 2025-03-05 20:23:02 +01:00
Owner
Copy link

To právě není úplně pravda. Někde nastala chyba, takže jsi dostal špatná data, a vracíš něco, co tomu neodpovídá.

To právě není úplně pravda. Někde nastala chyba, takže jsi dostal špatná data, a vracíš něco, co tomu neodpovídá.
ledoian approved these changes 2025-03-05 20:48:25 +01:00
ledoian left a comment
Owner
Copy link

Nevidím evidentní chyby, ale mám z toho kódu dost pocit, že sahat do toho bude strašně nepohodlné a náchylné na chyby, hlavně kvůli kopírovanému kódu, předávání magických číselných konstant a nezamýšlení se nad chybovými stavy.

IOW: kdyby mi hrozilo, že budu hlavní webař, tak budu hodně křičet, leč to mi nehrozí (prostě to dělat nebudu), takže si píšu jen černý puntík, na který kód sahat spíš nechci, a doporučení, jak se takového kódu vyvarovat.

(A trochu mám pocit, že když ten kód bude moc hnusný (nebo bude existovat jiný důvod odrazování webařů), tak se časem počet webařů sníží na jednoho, a až tenhle jeden webař odejde, tak skoro jediná cesta bude web přepsat celý, což je IMHO hlavně zmar úsilí…)

#sad-approve

Nevidím evidentní chyby, ale mám z toho kódu dost pocit, že sahat do toho bude strašně nepohodlné a náchylné na chyby, hlavně kvůli kopírovanému kódu, předávání magických číselných konstant a nezamýšlení se nad chybovými stavy. IOW: kdyby mi hrozilo, že budu hlavní webař, tak budu hodně křičet, leč to mi nehrozí (prostě to dělat nebudu), takže si píšu jen černý puntík, na který kód sahat spíš nechci, a doporučení, jak se takového kódu vyvarovat. (A trochu mám pocit, že když ten kód bude moc hnusný (nebo bude existovat jiný důvod odrazování webařů), tak se časem počet webařů sníží na jednoho, a až tenhle jeden webař odejde, tak skoro jediná cesta bude web přepsat celý, což je IMHO hlavně zmar úsilí…) `#sad-approve`
👍 1
personalni/templates/personalni/profil/export_lidi.html
@ -83,0 +101,4 @@
params += s + ","
}
}
params = params.slice(0, -1)
ledoian commented 2025-03-05 20:26:38 +01:00
Owner
Copy link

Nechceš použít fields.join(",")? Tohle mě neuráží, ale .join zní jakože to ušetří 5 řádků a dvě logické chyby…

Nechceš použít `fields.join(",")`? Tohle mě neuráží, ale `.join` zní jakože to ušetří 5 řádků a dvě logické chyby…
personalni/views.py
@ -170,2 +179,4 @@
def download_export_csv_only_first_step(request, type):
fields = getFieldsForExport(request)
if type == 3:
ledoian commented 2025-03-05 20:36:06 +01:00
Owner
Copy link

Uhh, a když type != 3, tak se stane co? A co je vůbec _only_first_step sémanticky?

Uhh, a když `type != 3`, tak se stane co? A co je vůbec `_only_first_step` sémanticky?
personalni/views.py
@ -192,3 +210,4 @@
name = str(soustredeni).replace(" ", "_") + "_organizatori_soustredeni.csv"
response['Content-Disposition'] = 'attachment; filename="' + name + '"'
return response
if type == PrvniTypExportu.SOUSTREDENI_UCASTNICI.value:
ledoian commented 2025-03-05 20:35:31 +01:00
Owner
Copy link

Tyhle ify vypadají dost zběsile. Chápu správně, že se prakticky jen mění, jaký QuerySet předhodíme dataOsobCsvResponse a pak možná filename?

Tyhle `if`y vypadají dost zběsile. Chápu správně, že se prakticky jen mění, jaký QuerySet předhodíme `dataOsobCsvResponse` a pak možná `filename`?
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

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

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

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?