front: meilleur UX pour supprimer ses bins #147
Comments
Mesteery
added
help wanted
Mesteery
changed the title
|
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 ( |
|
|
|
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. |
|
Oui, par contre, personnellement, je trouve IDB un peu overkill pour cet usage. |
|
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. |
|
Comme tu veux, pour moi ça m'est égal. |
|
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 |
The bin deletion via the interface was completly impratical. From now on, the token storage is done automatically. Indeed, a Service Worker is registered to intercept in particular the request of the form and the request of the final page (redirection). This allows to associate the token with the bin ID, which is not known in advance. In the background, the IndexedDB database is used via the `localforage` abstraction, which allows to easily store key-value in the same way as the `localStorage` (which is not available in a Service Worker). This feature is only available for the clients with JavaScript enabled. Closes: readthedocs-fr#147.
The bin deletion via the interface was completly impratical. From now on, the token storage is done automatically. Indeed, a Service Worker is registered to intercept in particular the request of the form and the request of the final page (redirection). This allows to associate the token with the bin ID, which is not known in advance. In the background, the IndexedDB database is used via the `localforage` abstraction, which allows to easily store key-value in the same way as the `localStorage` (which is not available in a Service Worker). This feature is only available for the clients with JavaScript enabled. Closes: readthedocs-fr#147.
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 #116 (comment)
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é.
The text was updated successfully, but these errors were encountered: