search

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

fix(fetch): use get_credit_accounts from API #85

Closed nmathey closed 1 year ago

nmathey commented 1 year ago

Description

Use get_credit_accounts function from API

Actions

lasconic commented 1 year ago

ça aiderait si je mettais l'api sur pip ? J'ai jamais fait mais je peux regarder dans les prochaines semaines.

nmathey commented 1 year ago

ça aiderait si je mettais l'api sur pip ? J'ai jamais fait mais je peux regarder dans les prochaines semaines.

Cela aiderait pour sûr les plus débutant d'entre nous découvrant ton API mais pour ce projet je laisse répondre @MadeInPierre ^^

MadeInPierre commented 1 year ago

Hello !

@lasconic J'avais passé un moment à apprendre comment faire sans que finary_api soit sur pip (via des git submodules) donc en soi tout va bien, mais ce serait chouette et plus simple (même pour moi) d'utiliser finary_api via pip ^^ Surtout si les versions évoluent souvent. En utilisant poetry (un outil parmi plusieurs), la publication sur pip est vraiment simple, ça ressemble en gros à :

pip install poetry
poetry init  # suivre ses questions
poetry add ... # ajouter les dépendances de finary_api
poetry publish

Ce serait quand même joli que n'importe qui puisse faire : 

pip install finary_api
python -m finary_api signin

Finary devrait vraiment fournir ça lui-même, tu bosses gratuitement pour eux :sweat_smile:

@nmathey Je t'avoue être un peu perdu avec ce problème :see_no_evil: la version de finary_api, même chez toi, n'aurait pas dû changer... donc pour l'instant on oublie https://github.com/lasconic/finary/pull/68, et dans ce cas le problème lié à e["account"]["name"] devrait venir d'ailleurs ? Peut-être pourrait-on chercher comme la dernière fois si ce n'est pas une histoire de RealT ou autre ?

Si ça te semble logique, pourrais-tu re-partager une réponse de l'API qui fait planter finalynx en joignant un fichier ici ? Merci !

En tout cas pour l'utilisation de ff.get_credit_accounts(session) c'est nickel ^^ j'attends la décision de @lasconic (pip ou pas) pour savoir comment j'update finary_api dans ce projet avant de merge.

lasconic commented 1 year ago

@MadeInPierre T'aurais demandé ! Je regarde dès que possible pour le mettre sur pip. Je pense que je le renommerai en finary_unofficial_api par contre...

Si vous avez des idées pour tester le projet... j'ai émis l'idée d'avoir accès à un compte de "demo" sur Finary sans succès. C'était dans un commentaire, je vais créer un post. Ça aiderait de pouvoir avoir un compte avec tous les types d'actifs, synchro auto ou manuel (même si ça sync pas vraiment derrière) non ?

lasconic commented 1 year ago

Go comment here : https://community.finary.com/t/dev-centraliser-les-posts-compte-de-demo/7461

MadeInPierre commented 1 year ago

A l'époque je ne savais même pas si j'allais perdre intérêt de continuer ce projet après 1 semaine :upside_down_face: Pas de soucis j'en ai profité pour apprendre !

Super pour pip, en effet un finary_api_unofficial serait bien (faudrait que Finary réserve le nom finary_api vite). Pour le compte démo, j'en ai vraiment besoin aussi, ça m'arrive assez souvent d'avoir des issues de gens sur des classes d'actifs que je n'ai même pas (coucou @nmathey ^^) et bon courage pour les aider...

J'ai hésité à créer un faux compte sur Finary pour y ajouter plein de lignes aléatoires sur toutes les classes d'actif, puis de mettre les infos de connexion dans les secrets de ce repo GitHub pour que le CI/CD teste que les intégrations fonctionnent. Mais même si ce serait un début, je ne pourrais pas prévoir tous les cas de réponse d'API possibles... pour l'instant mon CI/CD vérifie juste que Finalynx démarre sans contacter Finary :sweat_smile:

nmathey commented 1 year ago

Si ça te semble logique, pourrais-tu re-partager une réponse de l'API qui fait planter finalynx en joignant un fichier ici ? Merci !

Tout aussi perdu que toi :p Ci-joint la réponse API ce type: j'ai l'impression que "Account" n'est pas présent à chaque fois...

credits_acc.txt

MadeInPierre commented 1 year ago

Merci, en effet pas de account et pourtant ça marche pour d'autres personnes... bon, du coup je merge cette version (je viens de mettre à jour l'API en passant à finary_uapi dans #94) et on verra si quelqu'un ouvre une issue dans 2 jours ^^