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
64 stars 13 forks source link

fix(fetch): reviewed RealT token lookup by 'contractAddress'/'uuid' instead of 'symbol' #119

Closed nmathey closed 1 year ago

nmathey commented 1 year ago

Description

Reviewed RealT token lookup by 'contractAddress'/'uuid' instead of 'symbol' to avoid name mismatch between blockchain and RealT API response

Actions

MadeInPierre commented 1 year ago

Nice ! Il y a moyen de résoudre #120 en même temps par hasard ? Si tu me dis que cette PR est terminée je m'occupe des conflits + merge :slightly_smiling_face:

nmathey commented 1 year ago

@MadeInPierre Euh avec nos modifs en parallèle je ne suis pas sur de mon coup du coup maintenant :) Tu peux vérifier par rapport aux changements que tu as apportés entre temps?

nmathey commented 1 year ago

Nice ! Il y a moyen de résoudre #120 en même temps par hasard ? Si tu me dis que cette PR est terminée je m'occupe des conflits + merge 🙂

Yes ok je fais ca de suite

MadeInPierre commented 1 year ago

Joli bazar GitHub des fois, mais ça a un côté fun :slightly_smiling_face: Je reviens dans 1h sorry

nmathey commented 1 year ago

@MadeInPierre devrait être bon après avoir bien galéré avec Git :') Par contre je ne comprends pas pourquoi le "Semantic Release / pre-commit (pull_request)" ne passe pas.

Je vais me coucher la ca commence à faire tot/tard ici :D Au pire je regarderai de nouveau demain.

nmathey commented 1 year ago

Attention pour le #120 je pars du principe que les RealT Token resterons toujours en USD donc je force l'affichage à $

MadeInPierre commented 1 year ago

Merci @nmathey ! Et bon repos bien mérité :') J'ai fait un poil de ménage et corrigé quelques bugs mineurs en lien.

C'est "normal" pour le fail, car j'ai récemment ajouté un test dans GitHub Actions qui teste si Finalynx a bien accès à Finary API et récupère correctement les lignes. Ca fonctionne bien, sauf quand c'est quelqu'un qui crée une PR depuis un fork... (les forks n'ont pas accès aux secrets sauvegardés dans ce repo et les checks tournent depuis le fork)

Il faudra que je voie avec @lasconic car il s'y connaît peut-être mieux dans tout ça :slightly_smiling_face:

En attendant c'est nickel, je merge !

MadeInPierre commented 1 year ago

(@nmathey pour la prochaine, mieux vaut créer une nouvelle branche qui part de main (à jour) à chaque PR pour éviter les délires de merge :p tu peux supprimer celle-ci chez toi)

lasconic commented 1 year ago

Normalement, une PR tourne sur le repo sur lequel elle a été faite, elle a donc accès aux secrets.