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

Upgrade odevzdavatka #30

Merged
zelvuska merged 37 commits from upgrade_odevzdavatka into master 2023-06-19 20:48:30 +02:00
Conversation 3 Commits 37 Files changed 15 +2 -2
2 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit c64294099e - Show all commits

2
odevzdavatko/urls.py
View file

@ -20,7 +20,7 @@ from . import views
urlpatterns = [
path('org/add_solution', org_required(views.PosliReseniView.as_view()), name='seminar_vloz_reseni'),
path('resitel/nahraj_reseni', resitel_required(views.NahrajReseniNadproblemView.as_view()), name='seminar_nahraj_reseni'),
path('resitel/nahraj_reseni', resitel_required(views.NahrajReseniRozcestnikTematekView.as_view()), name='seminar_nahraj_reseni'),
path('resitel/nahraj_reseni/<int:nadproblem_id>/', resitel_required(views.NahrajReseniView.as_view()), name='seminar_nahraj_reseni'),
path('resitel/odevzdana_reseni/', resitel_or_org_required(views.PrehledOdevzdanychReseni.as_view()), name='seminar_resitel_odevzdana_reseni'),

2
odevzdavatko/views.py
View file

@ -399,7 +399,7 @@ class PosliReseniView(LoginRequiredMixin, FormView):
return data
class NahrajReseniNadproblemView(LoginRequiredMixin, ListView):
class NahrajReseniRozcestnikTematekView(LoginRequiredMixin, ListView):
zelvuska marked this conversation as resolved Outdated
ledoian commented 2023-06-19 19:09:11 +02:00
Outdated
Review
Copy link

Možná lepší pojmenovat ve stylu NahrajReseniRozcestnikTematekView nebo NahrajReseniVyberHlavnihoProblemuView, nebo dokonce OdevzdavatkoVyberTematkaView? Tady se žádné řešení nenahrává, a technicky to ani není nadproblém, ale hlavní problém (bo i nadproblém může mít nadproblém, teoreticky).

Možná lepší pojmenovat ve stylu `NahrajReseniRozcestnikTematekView` nebo `NahrajReseniVyberHlavnihoProblemuView`, nebo dokonce `OdevzdavatkoVyberTematkaView`? Tady se žádné řešení nenahrává, a technicky to ani není nadproblém, ale hlavní problém (bo i nadproblém může mít nadproblém, teoreticky).
model = m.Problem
template_name = 'odevzdavatko/nahraj_reseni_nadproblem.html'