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

Podezřelé semináře (#1465) #65

Merged
zelvuska merged 27 commits from podezrele-seminare into master 2024-11-05 20:31:55 +01:00
Conversation 9 Commits 27 Files changed 102 +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 ac6c41cc88 - Show all commits

2
soustredeni/models.py
View file

@ -75,7 +75,7 @@ class Soustredeni(SeminarModelBase):
def verejne_url(self): def verejne_url(self):
#return reverse('seminar_soustredeni', kwargs={'pk': self.id}) #return reverse('seminar_soustredeni', kwargs={'pk': self.id})
return reverse('soustredeni_probehlo') return reverse('soustredeni_seznam')
@reversion.register(ignore_duplicates=True) @reversion.register(ignore_duplicates=True)

2
soustredeni/urls.py
View file

@ -7,7 +7,7 @@ urlpatterns = [
path( path(
'probehlo/', 'probehlo/',
views.SoustredeniListView.as_view(), views.SoustredeniListView.as_view(),
name='soustredeni_probehlo' name='soustredeni_seznam'
zelvuska marked this conversation as resolved Outdated
ledoian commented 2024-11-03 00:52:55 +01:00
Outdated
Review
Copy link

S tímhle přejmenováním spíš nesouhlasím. To, že se na webu seznam soustředění vyskytuje v „Soustředění → Proběhlo“ je podle mě UI detail a z pohledu vývojáře chci spíš vědět, že adresa vede na seznam soustředění (nebo přehled soustředění).

S tímhle přejmenováním spíš nesouhlasím. To, že se na webu seznam soustředění vyskytuje v „Soustředění → Proběhlo“ je podle mě UI detail a z pohledu vývojáře chci spíš vědět, že adresa vede na seznam soustředění (nebo přehled soustředění).
zelvuska marked this conversation as resolved
ledoian commented 2024-11-05 16:33:56 +01:00
Review
Copy link
$ git grep probehlo
data/sitetree.json:                     "url": "soustredeni_probehlo",
personalni/templates/personalni/profil/orgorozcestnik.html:     <li><a href="{% url 'soustredeni_probehlo' %}">proběhlá soustředění</a>
``` $ git grep probehlo data/sitetree.json: "url": "soustredeni_probehlo", personalni/templates/personalni/profil/orgorozcestnik.html: <li><a href="{% url 'soustredeni_probehlo' %}">proběhlá soustředění</a> ```
), ),
path( path(
'<int:soustredeni>/', '<int:soustredeni>/',