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

Rozstřílení seminářové aplikace #60

Merged
zelvuska merged 19 commits from split into master 2024-10-22 21:27:21 +02:00
Conversation 2 Commits 19 Files changed 107 +2365 -1623
zelvuska commented 2024-08-06 02:37:33 +02:00
Owner
Copy link

Přemýšlím, jestli tohle vůbec reviewovat, jestli nedává víc smysl v tom rovnou začít programovat dál a případně pak řešit „tohle jsem nemohl najít“ jednotlivě.

Speciálně tohle by asi chtělo nasadit dříve, než se noví webaři pustí do kódění (aby si nezvykli na starý řád věcí; je lepší řešit „tohle jsem nemohl najít“ než „tohle jsem nemohl najít, protože jsem to hledal tam, kde to vždycky bývalo“).

Přemýšlím, jestli tohle vůbec reviewovat, jestli nedává víc smysl v tom rovnou začít programovat dál a případně pak řešit „tohle jsem nemohl najít“ jednotlivě. Speciálně tohle by asi chtělo nasadit dříve, než se noví webaři pustí do kódění (aby si nezvykli na starý řád věcí; je lepší řešit „tohle jsem nemohl najít“ než „tohle jsem nemohl najít, protože jsem to hledal tam, kde to vždycky bývalo“).
zelvuska added 16 commits 2024-08-06 02:37:34 +02:00
zelvuska requested review from ledoian 2024-09-21 23:39:00 +02:00
ledoian commented 2024-10-22 19:45:04 +02:00
Owner
Copy link

Nechceme v tvorbě pojmenovat views_all.py jako views.py? Je tam ještě něco dalšího někde?

Nechceme v tvorbě pojmenovat `views_all.py` jako `views.py`? Je tam ještě něco dalšího někde?
ledoian reviewed 2024-10-22 19:46:50 +02:00
various/templates/various/pracuje_se.html Outdated
@ -0,0 +10,4 @@
<p>
Na této stránce velmi intenzivně pracujeme.
Za dočasnou nedostupnost se omlouváme.
Zkuste přejít na <a href="/static">titulní stránku</a>
ledoian commented 2024-10-22 19:46:50 +02:00
Owner
Copy link

ehh? FIXME?

ehh? FIXME?
zelvuska marked this conversation as resolved
ledoian commented 2024-10-22 19:58:27 +02:00
Owner
Copy link

Rozdělení various/views/pomocne.py a various/views/final.py mi nedává moc smysl, mám pocit, že v obou jsou dost random věci (vč. věcí které spíš patří do nějakých utils.py jako histogram()…) a tohle dělení mi přijde dost ortogonální na to, že ostatní jména souborů v various/views/ odpovídají použití té věci… Možná prostě sjednotit do various/views/other.py?

Rozdělení `various/views/pomocne.py` a `various/views/final.py` mi nedává moc smysl, mám pocit, že v obou jsou dost random věci (vč. věcí které spíš patří do nějakých `utils.py` jako `histogram()`…) a tohle dělení mi přijde dost ortogonální na to, že ostatní jména souborů v `various/views/` odpovídají použití té věci… Možná prostě sjednotit do `various/views/other.py`?
zelvuska added 1 commit 2024-10-22 20:16:08 +02:00
zelvuska added 1 commit 2024-10-22 20:19:10 +02:00 
# Conflicts:
#	various/templates/various/titulnistrana/titulnistrana.html
zelvuska removed review request for ledoian 2024-10-22 20:40:34 +02:00
zelvuska added 1 commit 2024-10-22 21:23:27 +02:00
zelvuska merged commit 7108702e36 into master 2024-10-22 21:27:21 +02:00
zelvuska deleted branch split 2024-10-22 21:27:21 +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#60
No description provided.
Delete branch "split"

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?