MadeInPierre / finalynx

A minimalistic companion (CLI & web) to organize your investment portfolio, simulate its future, and reach your life goals.
https://finalynx.readthedocs.io
GNU General Public License v3.0
68 stars 13 forks source link

erreur sur date unlock #98

Closed sebfar9172 closed 1 year ago

sebfar9172 commented 1 year ago

Hello

J'ai cette enveloppe :

startupNothing=Envelope("Nothing", "STP", date_created=date(2022, 4, 6), date_unlock=date(2027, 4, 6))

cela me fait une erreur en m'indiquant que la date de création doit être inférieure à la date unlock mais c'est bien le cas car 2022 < 2027

je pense que c'est la règle suivante qui pose problème, en effet si on ne saisit pas de date_untax cela met date_created donc on a date_created < date_unlock < date_created ce qui ne fonctionne pas :)

self.date_created = date_created
self.date_unlock = date_unlock if date_unlock else date_created
self.date_untax = date_untax if date_untax else date_created

if not (self.date_created <= self.date_unlock <= self.date_untax):
    raise ValueError("Envelope dates must be ordered by created <= unlock <= untax")
MadeInPierre commented 1 year ago

Ah mince bien vu ! Le fix suivant est très simple et me semble correct :

self.date_untax = date_untax if date_untax else self.date_unlock

Ca te paraît bon ? Fin de journée, cerveau fatigué x)

sebfar9172 commented 1 year ago

En effet c'est le plus simple (Pour l'instant j'ai ajouté la date untax )

MadeInPierre commented 1 year ago

Ca marche je push de suite, si un jour ça te tente d'upload les contributions toi-même n'hésite pas, avec plaisir pour expliquer ^^

sebfar9172 commented 1 year ago

Ca marche je push de suite, si un jour ça te tente d'upload les contributions toi-même n'hésite pas, avec plaisir pour expliquer ^^

Yes cela sera plus simple

MadeInPierre commented 1 year ago

Au prochain bug que tu trouves si t'es chaud on se fera les étapes ensemble :grin: merci pour celui-ci !