search

vsvandelik / vzs-clenska-sekce

MIT License
0 stars 0 forks source link

Pridat u dluhu tlacitko Zaplatit #584

Closed vsvandelik closed 9 months ago

vsvandelik commented 10 months ago

Jen u dluhu, u odmeny vubec neevidujeme zaplaceni

vsvandelik commented 10 months ago

@papundekel Mas nejaky dobry napad jak na tohle? Ktery by nebyl moc narocny. Pokud mas, tak to prosim zkus, pokud ne, tak bychom tohle asi uz neimplementovali.

peter-facko commented 10 months ago

No napada mi:

  1. Pridat bool field manually_settled do modelu. Otazka je co znamena mat nastavene aj to aj fio transakciu
  2. Vytvorit novy model ktory bude mat len foreign key na transakciu. Nezmeni to ziadny model, ale je tam problem ze queries sa budu blbsie implementovat a ostava otazka toho "dvojiteho" settle.
  3. Vytvorit fio transakciu s ID 0 a na nu ukazat vsetky manualne zaplatene. Nevyhoda je len ze model nezodpoveda realite.

Ja som asi za 3. kedze to nemeni aktualnu logiku. Pripadne mozme este upravit model fio transakcie aby nebolo fio_id required. Este by tam bola otazka ako sa zachovat k tomu date_settled. Asi by bolo vhodne dat aktualny datum alebo dovolit manualne ho nastavit.

vsvandelik commented 10 months ago

No napada mi:

  1. Pridat bool field manually_settled do modelu. Otazka je co znamena mat nastavene aj to aj fio transakciu
  2. Vytvorit novy model ktory bude mat len foreign key na transakciu. Nezmeni to ziadny model, ale je tam problem ze queries sa budu blbsie implementovat a ostava otazka toho "dvojiteho" settle.
  3. Vytvorit fio transakciu s ID 0 a na nu ukazat vsetky manualne zaplatene. Nevyhoda je len ze model nezodpoveda realite.

Ja som asi za 3. kedze to nemeni aktualnu logiku. Pripadne mozme este upravit model fio transakcie aby nebolo fio_id required. Este by tam bola otazka ako sa zachovat k tomu date_settled. Asi by bolo vhodne dat aktualny datum alebo dovolit manualne ho nastavit.

Hele ta trojka je sice nejjednodušší, ale není hezká. A měnit model se mi už nechce vzhledem k blížícímu se odevzdání. Tedy navrhuji tento task NEimplementovat a zavřít. Pokud souhlasíš, prosím, zavři ho.

peter-facko commented 9 months ago

Okej, mas pravdu ze to nema aktualne dobre riesenie.