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

WIP: Prehlednejsi Hodnotitko (tabulka) #31

Draft
ledoian wants to merge 2 commits from prehlednejsi_hodnotitko into master
pull from: prehlednejsi_hodnotitko
merge into: mam:master
Conversation 4 Commits 2 Files changed -
ledoian commented 2023-05-22 21:27:54 +02:00
Owner
Copy link

WIP

WIP
ledoian added 1 commit 2023-05-22 21:27:55 +02:00
ledoian added 10 commits 2023-05-22 23:58:22 +02:00 
Už není potřeba, v tabulce stejně jsou už přímo řešení.
Snad, ještě jsem to netestoval :-)
TODO: rebase squash
todo squash
Hnusně, ale přece…
ledoian added 1 commit 2023-05-23 00:28:38 +02:00
zelvuska changed title from Prehlednejsi Hodnotitko to Prehlednejsi Hodnotitko (tabulka) 2023-06-12 22:17:47 +02:00
zelvuska reviewed 2023-06-15 17:21:43 +02:00
odevzdavatko/forms.py Outdated
@ -196,6 +196,7 @@ class OdevzdavatkoTabulkaFiltrForm(forms.Form):
'reseni_od': terminy[-2] if rocnik is None else terminy[0],
'reseni_do': terminy[-1],
'neobodovane': False,
'barvicky': True,
zelvuska commented 2023-06-15 17:21:43 +02:00
Owner
Copy link

Pokud tohle má dělat, že 🎨? bude checked, pak to u mě nějak nefunguje…

Pokud tohle má dělat, že `🎨?` bude checked, pak to u mě nějak nefunguje…
zelvuska reviewed 2023-06-15 17:26:24 +02:00
odevzdavatko/views.py
@ -133,2 +135,3 @@
if resitel not in tabulka[problem]:
tabulka[problem][resitel] = SouhrnReseni(pocet_reseni=1, posledni_odevzdani=cas, body=body)
tabulka[problem][resitel] = [(reseni, hodnoceni)]
soucty[problem][resitel] = hodnoceni.body or 0 # Neobodované neřešíme
zelvuska commented 2023-06-15 17:26:24 +02:00
Owner
Copy link

A jak tedy poznám neobodované? To si je musím nechat vyjet zvlášť?

A jak tedy poznám neobodované? To si je musím nechat vyjet zvlášť?
zelvuska commented 2023-06-15 18:41:39 +02:00
Owner
Copy link

Á jasně, to je v součtu.

Á jasně, to je v součtu.
zelvuska marked this conversation as resolved
zelvuska reviewed 2023-06-15 17:30:44 +02:00
odevzdavatko/views.py Outdated
@ -150,3 +148,3 @@
for resitel in self.resitele:
dostal_body = False
resiteluv_radek = []
resiteluv_radek: list[tuple[Decimal,list[tuple[m.Resitel, m.Hodnoceni]]]] = [] # podle pořadí v self.problemy
zelvuska commented 2023-06-15 17:30:44 +02:00
Owner
Copy link

Tady má být m.Reseni, pokud se nepletu (místo m.Resitel).

Tady má být `m.Reseni`, pokud se nepletu (místo `m.Resitel`).
ledoian commented 2023-06-20 00:26:21 +02:00
Author
Owner
Copy link

Taky si myslím :-) Jen tak pro zábavu jsem zkusil na to poštvat typovou kontrolu (mypy v1.3.0), ale ta nic použitelného říct neumí, protože Django nemá typové anotace :-/

(Existuje nějaký komunitní projekt, který je přidává: https://github.com/typeddjango/django-stubs. Možná někdy prozkoumám…)

Taky si myslím :-) Jen tak pro zábavu jsem zkusil na to poštvat typovou kontrolu (mypy v1.3.0), ale ta nic použitelného říct neumí, protože Django nemá typové anotace :-/ (Existuje nějaký komunitní projekt, který je přidává: <https://github.com/typeddjango/django-stubs>. Možná někdy prozkoumám…)
zelvuska approved these changes 2023-06-15 17:31:50 +02:00
zelvuska left a comment
Owner
Copy link

LGTM

LGTM
odevzdavatko/views.py
@ -131,0 +127,4 @@
def pridej_reseni(resitel, hodnoceni):
problem = hodnoceni.problem
body = hodnoceni.body
cas = hodnoceni.reseni.cas_doruceni
zelvuska commented 2023-06-15 17:31:41 +02:00
Owner
Copy link

Body a cas jsou nepoužité. Jinak ten kód asi dává smysl.

Body a cas jsou nepoužité. Jinak ten kód asi dává smysl.
ledoian added 1 commit 2023-06-20 00:26:51 +02:00
ledoian commented 2024-05-21 21:58:13 +02:00
Author
Owner
Copy link

Tohle jde prostě zavřít? Vypadá to mergenutě…

Tohle jde prostě zavřít? Vypadá to mergenutě…
zelvuska commented 2024-05-21 21:59:23 +02:00
Owner
Copy link

Ale není ještě 100% funkční (nesvítí zaškrtnuto, přestože jsou zaplé).

Ale není ještě 100% funkční (nesvítí zaškrtnuto, přestože jsou zaplé).
zelvuska commented 2024-05-21 22:00:03 +02:00
Owner
Copy link

A myslím, že jsme se bavili nad lepšími barvami… (Třeba nebílými.)

A myslím, že jsme se bavili nad lepšími barvami… (Třeba nebílými.)
ledoian commented 2024-05-21 22:05:31 +02:00
Author
Owner
Copy link

Ha, vidíš, i forgor…

Ha, vidíš, i forgor…
zelvuska added 1 commit 2024-10-23 12:44:25 +02:00
zelvuska changed title from Prehlednejsi Hodnotitko (tabulka) to WIP Prehlednejsi Hodnotitko (tabulka) 2024-10-24 11:09:03 +02:00
zelvuska changed title from WIP Prehlednejsi Hodnotitko (tabulka) to WIP: Prehlednejsi Hodnotitko (tabulka) 2024-10-24 11:09:13 +02:00
zelvuska added 1 commit 2024-10-24 11:09:33 +02:00
This pull request is marked as a work in progress.
This branch is out-of-date with the base branch
View command line instructions

Checkout

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

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 prehlednejsi_hodnotitko
git checkout prehlednejsi_hodnotitko
git rebase master
git checkout master
git merge --ff-only prehlednejsi_hodnotitko
git checkout prehlednejsi_hodnotitko
git rebase master
git checkout master
git merge --no-ff prehlednejsi_hodnotitko
git checkout master
git merge --squash prehlednejsi_hodnotitko
git checkout master
git merge --ff-only prehlednejsi_hodnotitko
git checkout master
git merge prehlednejsi_hodnotitko
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#31
No description provided.
Delete branch "prehlednejsi_hodnotitko"

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?