Reforma pohlaví #49

Merged
zelvuska merged 5 commits from reforma_pohlavi into master 4 months ago
ledoian commented 4 months ago
Owner

Přepsal jsem, jak se chováme k pohlaví. Osobně mi přijde, že se spíš hodí enum, než to mlátit booleanem (věděli jste, že máme v testdatech pohlaví řešitelů přesně otočená, a to od nepaměti? :-D). Kromě toho, pro web pohlaví není skoro nikde použité: kromě toho, že je generujeme v testdatech a vyskytují se v CSV exportu, jsou v AESOP exportu (jehož budoucnost je nejasná, ale whatever) a formátoval se podle toho mail o došlém řešení (to jsem přepsal do přít. času a tím odpadla potřeba genderovat).

O tom, jak se má pole v modelu jmenovat nebo jestli pohlaví nezahodit úplně klidně můžeme diskutovat, asi nemám moc silné názory…

Jen bych byl pro, aby to nebyl boolean – btw, používám Arch do binárních genderových krabiček sám moc nepasuji :-)

Přepsal jsem, jak se chováme k pohlaví. Osobně mi přijde, že se spíš hodí enum, než to mlátit booleanem (věděli jste, že máme v testdatech pohlaví řešitelů přesně otočená, a to od nepaměti? :-D). Kromě toho, pro web pohlaví není skoro nikde použité: kromě toho, že je generujeme v testdatech a vyskytují se v CSV exportu, jsou v AESOP exportu (jehož budoucnost je nejasná, ale whatever) a formátoval se podle toho mail o došlém řešení (to jsem přepsal do přít. času a tím odpadla potřeba genderovat). O tom, jak se má pole v modelu jmenovat nebo jestli pohlaví nezahodit úplně klidně můžeme diskutovat, asi nemám moc silné názory… Jen bych byl pro, aby to nebyl boolean – btw, ~~používám Arch~~ do binárních genderových krabiček sám moc nepasuji :-)
ledoian added 1 commit 4 months ago
ledoian added 1 commit 4 months ago
Poster
Owner

Uh, po mergi fakt hodně nevím, jak moc tenhle PR je funkční :-/

Uh, po mergi fakt hodně nevím, jak moc tenhle PR je funkční :-/
ledoian added 2 commits 4 months ago
ledoian added 1 commit 4 months ago
Poster
Owner

Tak teď už to vypadá funkčněji…

Tak teď už to vypadá funkčněji…
Owner

Za mě to vypadá OK. Zkusil bych to nasadit na testweb a podívat se, co to provede…

Za mě to vypadá OK. Zkusil bych to nasadit na testweb a podívat se, co to provede…
Poster
Owner

Na testwebu to je, data vypadají v pořádku. Můžu zkusit nahrát řešení, ale problém nečekám… (AESOPí API nastudované nemám, tak si po něm povídat spíš nezvládnu…)

Na testwebu to je, data vypadají v pořádku. Můžu zkusit nahrát řešení, ale problém nečekám… (AESOPí API nastudované nemám, tak si po něm povídat spíš nezvládnu…)
Owner

Přišly errory z testwebu…

Přišly errory z testwebu…
Poster
Owner

Přišly errory z testwebu…

Bo jsem nešika a načítám stránku během deploye. Ignorovat, tentýž požadavek o chvíli později prošel :-)

> Přišly errory z testwebu… Bo jsem nešika a načítám stránku během deploye. Ignorovat, tentýž požadavek o chvíli později prošel :-)
Poster
Owner

Jen čím dál víc přemýšlím, jestli to prostě nezahodit úplně. Mít v databázi kolonku jen kvůli jedné volitelné položce v AESOPím exportu (a pak aby bylo co zobrazovat na /stav :-P) je možná trochu zbytečné…

Jen čím dál víc přemýšlím, jestli to prostě nezahodit úplně. Mít v databázi kolonku jen kvůli jedné volitelné položce v AESOPím exportu (a pak aby bylo co zobrazovat na `/stav` :-P) je možná trochu zbytečné…
Owner

Tak občas řešitelům posíláme e-maily (například na soustředění).

Možná se nám to také hodí vědět kvůli pokojům na soustředění a také kvůli zdravotním věcem. Ale je pravda, že tohle by mohlo být ve zdravotnickém dotazníku.

Tak občas řešitelům posíláme e-maily (například na soustředění). Možná se nám to také hodí vědět kvůli pokojům na soustředění a také kvůli zdravotním věcem. Ale je pravda, že tohle by mohlo být ve zdravotnickém dotazníku.
Owner

Já bych to tam prozatím mergnul takhle…

Já bych to tam prozatím mergnul takhle…
Poster
Owner

Tak občas řešitelům posíláme e-maily (například na soustředění).

Možná se nám to také hodí vědět kvůli pokojům na soustředění a také kvůli zdravotním věcem. Ale je pravda, že tohle by mohlo být ve zdravotnickém dotazníku.

Jenže je velmi nepraktické komunikovat, jestli se zrovna ptáme „jak se vnitřně cítíš?“ (tj. na co se ptá AESOP asi), „jak tě máme oslovovat?“ (pro maily ap.) nebo „jaké jsou tvé hladiny hormonů?“ (nebo co přesně je u lidí ten zdravotně relevantní faktor). A tyhle otázky nemusí mít stejnou odpověď…

To je ostatně jeden z důvodů, proč to přejmenovávám z „pohlaví“ na „oslovení“ – aby se to ptalo na tu otázku, která web zajímá nejvíc.

Já bych to tam prozatím mergnul takhle…

OK.

> Tak občas řešitelům posíláme e-maily (například na soustředění). > > Možná se nám to také hodí vědět kvůli pokojům na soustředění a také kvůli zdravotním věcem. Ale je pravda, že tohle by mohlo být ve zdravotnickém dotazníku. Jenže je velmi nepraktické komunikovat, jestli se zrovna ptáme „jak se vnitřně cítíš?“ (tj. na co se ptá AESOP asi), „jak tě máme oslovovat?“ (pro maily ap.) nebo „jaké jsou tvé hladiny hormonů?“ (nebo co přesně je u lidí ten zdravotně relevantní faktor). A tyhle otázky nemusí mít stejnou odpověď… To je ostatně jeden z důvodů, proč to přejmenovávám z „pohlaví“ na „oslovení“ – aby se to ptalo na tu otázku, která web zajímá nejvíc. > Já bych to tam prozatím mergnul takhle… OK.
zelvuska merged commit e06080ae31 into master 4 months ago
zelvuska deleted branch reforma_pohlavi 4 months ago
The pull request has been merged as e06080ae31.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.