search

readthedocs-fr / bin-server

Un outil pour héberger des snippets de code et les partager via une URL.
https://bin.readthedocs.fr
MIT License
14 stars 15 forks source link

front: meilleur UX pour supprimer ses bins #147

Open Mesteery opened 3 years ago

Mesteery commented 3 years ago

Vu la complexité énorme de cette PR. Je vais enlever l'enregistrement du token dans le localStorage. On discutera de ça dans une issue. Alors, l'utilisateur devrait spécifier manuellement le token.

Originally posted by @Mesteery in https://github.com/readthedocs-fr/bin-server/issues/116#issuecomment-822543760

Actuellement la suppression d'un bin via l'interface est vraiment pas pratique, l'UX est mauvaise étant donné que nous devons manuellement spécifier le token dans l'url via ?token=.... Il faudrait trouver un moyen permettant d’enregistrer ce token quelque part (dans le navigateur bien sûr). En outre, il faudrait un "hook" qui enregistre le token d'un bin après sa soumission. Iil y a déjà quelques essaies et brouillons, qui toutefois fonctionnent, mais qui sont bugés en quelque sorte (cf. #116).

D'ailleurs, une idée proposé par @fusetim est d'utiliser un service worker. Malheureusement, ce n'est pas possible (le localStorage n'est pas accessible dans un SW).

Un bonus serait de supprimer le token automatiquement une fois le bin expiré.

fusetim commented 3 years ago

Il n'est pas forcément nécessaire d'avoir le localStorage depuis les SW pour le faire, il serait possible de manipuler le cache pour les y sauvegarder dedans (Cache.put()). Je ne sais pas si c'est particulièrement idiomatique mais ça pourrait fonctionner.

Mesteery commented 3 years ago

C'est pas du tout adapté à ce genre d'usage. Quoi que, ça m'a pas l'air déconnant.

fusetim commented 3 years ago

Bon après un peu de recherche, c'est IndexedDB qui est préféré pour cette usage mais c'est aussi plus compliqué à utiliser.

Mesteery commented 3 years ago

Oui, par contre, personnellement, je trouve IDB un peu overkill pour cet usage.

fusetim commented 3 years ago

Pour moi aussi mais théoriquement c'est là où il faudrait les sauvegarder, mais dans notre cas, utiliser l'interface Cache me semble pas être une aberration.

Mesteery commented 3 years ago

Comme tu veux, pour moi ça m'est égal.

Julien00859 commented 3 years ago

Sinon, on big cookie boi mais essayons avec le cache dans un premier temps, on n'y avait pas pensé à l'époque où on a designer la suppression