-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
refactor: toasted and authenticated config params for API methods #464
Conversation
✅ Deploy Preview for meteo-france ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for ecospheres ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
ac817ee
to
c5c9282
Compare
f8def37
to
913b5f5
Compare
Ça fonctionne dans le sens où il n'y a plus d'erreur et l'URL n'est pas internalisée. Mais je me demande si on devrait signaler l'erreur dans ce cas ? |
Je me suis posé la question mais pas sûr de l'intérêt de rajouter cette complexité : on essaie d'être plus intelligent que l'usager et en plus on l'embête parce qu'on y arrive pas ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je me suis posé la question mais pas sûr de l'intérêt de rajouter cette complexité : on essaie d'être plus intelligent que l'usager et en plus on l'embête parce qu'on y arrive pas ?
Pas faux. Cela dit une fois habitué, l'usagé risque de s'attendre à ce fonctionnement et être surpris lorsque ça ne fonctionne pas. J'anticipe peut-être trop... On peut aussi attendre que le problème remonte (si jamais ça remonte).
Introduit les paramètres suivants pour chaque appel d'API :
toasted
: est-ce qu'un toast doit être affiché en cas d'erreur ?authentificated
: est-ce que la clé d'API doit être envoyée si elle existe ?Ces changements permettent :
On peut ainsi traiter les deux
FIXME
:cloned_from
non trouvé (parce que supprimé probablement)Cela ouvre aussi la possibilité de traiter ecolabdata/ecospheres#217 et ecolabdata/ecospheres#221.