TableauBits / Mellotron

Static archive of Matbay songs
https://mellotron.pages.dev
MIT License
1 stars 0 forks source link

Réinitialiser les paramètres du Local Storage #49

Closed hvdbm closed 2 years ago

hvdbm commented 2 years ago

Description

Ajout d'un bouton pour réinitialiser les paramètres sauvegardés dans le local storage. Cela permet de plus facilement travailler sur différentes branches avec des paramètres de formats différents et de revenir à état stable en cas de problème.

:tada: Quality of life

Dépendance issues/pull request

Checklist

À être vérifié par les reviewers:

cloudflare-workers-and-pages[bot] commented 2 years ago

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: f0ba4c1
Status: ✅  Deploy successful!
Preview URL: https://24341427.mellotron.pages.dev
Branch Preview URL: https://reset-local-storage.mellotron.pages.dev

View logs

rakruk commented 2 years ago

Pour l'instant ça ressemble à ça pour moi: image Je propose de modifier le scope du ticket pour ajouter un try/catch à l'appel de tout localStorage.get() notamment dans le constructeur de TableViewerComponent et mettre l'appel à reset dans le catch.

J'ai également créé une tache #50 sur le board pour déplacer les appels à localstorage.get() dans un DataService

hvdbm commented 2 years ago

@rakruk j'ai vu l'erreur, j'ai enlevé la demande de review pour essayer de corriger ça.

rakruk commented 2 years ago

D'ailleurs @hvdbm c'est pour ça que je proposais les valeurs par défauts, je l'ai un peu venu venir 😉 https://github.com/TableauBits/Mellotron/pull/31#discussion_r906713844

hvdbm commented 2 years ago

@rakruk j'ai laissé un commentaire sur ta suggestion, mais tu m'as jamais répondu et tu as ensuite validé directement la PR 😢 https://github.com/TableauBits/Mellotron/pull/31#discussion_r906713844

hvdbm commented 2 years ago
Capture d’écran, le 2022-07-25 à 14 58 17

C'est sûr que si on est pas capable d'appeler une fonction ça va beaucoup moins bien marcher 🤦