Framadate #88

Closed
ticvac wants to merge 1 commit from framadate into master
Owner

No, ve čtvrtek jsem ze srandy napsal ten framadate... říkal jsem si že ho sem nebudu házet, ale bylo mi ho trochu líto... tak tu je;)

No, ve čtvrtek jsem ze srandy napsal ten framadate... říkal jsem si že ho sem nebudu házet, ale bylo mi ho trochu líto... tak tu je;)
ticvac added 1 commit 2025-01-26 18:29:17 +01:00
zelvuska reviewed 2025-01-26 19:21:14 +01:00
@ -64,1 +64,4 @@
path('sifrovacka/', include('sifrovacka.urls')),
# Framadate
path('', include('framadate.urls')),
Owner

Proč '' a ne 'framadate/?

Proč `''` a ne `'framadate/`?
zelvuska reviewed 2025-01-26 19:22:32 +01:00
@ -0,0 +70,4 @@
def delete_framadate(request, pk):
framadate = Framadate.objects.get(pk=pk)
framadate.delete()
return HttpResponseRedirect("/framadate")
Owner

reverse(...)

`reverse(...)`
zelvuska reviewed 2025-01-26 19:25:44 +01:00
@ -0,0 +73,4 @@
return HttpResponseRedirect("/framadate")
def addContextForFramadate(context, **kwargs):
context['framadate'] = Framadate.objects.get(pk=kwargs['pk'])
Owner

A když neexistuje? (get_object_or_404)

A když neexistuje? (`get_object_or_404`)
zelvuska reviewed 2025-01-26 19:28:03 +01:00
@ -0,0 +35,4 @@
ordering = ['day', 'time']
class Choice(models.IntegerChoices):
UNKNOWN = 0
Owner

Proč to nemá popisky? (Např. UNKNOWN = 0, "Nevím")

Proč to nemá popisky? (Např. `UNKNOWN = 0, "Nevím"`)
zelvuska reviewed 2025-01-26 19:30:11 +01:00
@ -0,0 +5,4 @@
class Framadate(models.Model):
name = models.TextField()
"isRelevant if all dates are in the future"
Owner

Myslím si, že framadaty většinou bývají relevatntní úplně jiným způsobem. (Jakože většinou hlasování končí dávno před prvním termínem, ale občas končí i po něm.)

Myslím si, že framadaty většinou bývají relevatntní úplně jiným způsobem. (Jakože většinou hlasování končí dávno před prvním termínem, ale občas končí i po něm.)
Owner

Jak jsem říkal na schůzi. Přijde mi zbytečné přidávat tuto funkcionalitu do webu, protože webařům akorát přidělá práci s údržbou a existuje rozumná alternativa. Nehledě na to, že se orgové budou muset tento framadate naučit používat (orgům zrovna dvakrát nejde naučit se používat nějaký software), a nemůžou ho použít nikde mimo M&M (to mi přijde jako důležité minus, prostě proč bych měl používat něco, co stejně nemůžu použít mimo M&M, když existuje veřejná alternativa).

Jak jsem říkal na schůzi. Přijde mi zbytečné přidávat tuto funkcionalitu do webu, protože webařům akorát přidělá práci s údržbou a existuje rozumná alternativa. Nehledě na to, že se orgové budou muset tento framadate naučit používat (orgům zrovna dvakrát nejde naučit se používat nějaký software), a nemůžou ho použít nikde mimo M&M (to mi přijde jako důležité minus, prostě proč bych měl používat něco, co stejně nemůžu použít mimo M&M, když existuje veřejná alternativa).
zelvuska reviewed 2025-01-26 19:35:23 +01:00
@ -0,0 +1,3 @@
from django.contrib import admin
# Register your models here.
Owner

A ty věci nejsou upravitelné v adminu proč?

A ty věci nejsou upravitelné v adminu proč?
zelvuska reviewed 2025-01-26 19:37:19 +01:00
@ -0,0 +14,4 @@
overflow-x: hidden;
}
.table {
Owner

To není zrovna popisný název třídy…

To není zrovna popisný název třídy…
Owner

Také odsazení použité v html souborech není moc kompatibilní s M&M webem…

Obdobně v M&M píšeme česky (nebo alespoň většinu).

Také odsazení použité v html souborech není moc kompatibilní s M&M webem… Obdobně v M&M píšeme česky (nebo alespoň většinu).
zelvuska reviewed 2025-01-26 19:42:23 +01:00
@ -0,0 +3,4 @@
{% load static %}
<link rel="stylesheet" href="{% static 'framadate/style.css' %}">
<link rel="stylesheet" href="{% static 'framadate/new_framadate.css' %}">
<!-- fonts -->
Owner

Proč?

Proč?
zelvuska closed this pull request 2025-02-26 14:30:02 +01:00

Pull request closed

Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
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#88
No description provided.