Skip to content
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

Feature request : gestion de plusieurs persos sur un même device #14

Open
rdugau opened this issue Feb 15, 2023 · 3 comments
Open

Feature request : gestion de plusieurs persos sur un même device #14

rdugau opened this issue Feb 15, 2023 · 3 comments
Assignees
Labels
Feature request User submitted feature request

Comments

@rdugau
Copy link

rdugau commented Feb 15, 2023

Grosse feature ;-)
L'idée est de pouvoir gérer plusieurs perso sur un même device

Cette fonctionnalité peut être utile pour le PJ pour gérer plusieurs persos mais aussi pour le MJ afin de pouvoir voir tous les persos de sa partie en cours.

(perso je verrais bien un système d'onglet ou de burger menu pour basculer d'un perso à l'autre)

PS :
Comment fait-on pour tagger directement une issue "Feature request" ?

@rdugau rdugau changed the title Feature request : gestion de plusieurs perso sur un même device Feature request : gestion de plusieurs persos sur un même device Feb 15, 2023
@GoOz GoOz self-assigned this Feb 15, 2023
@GoOz GoOz added the Feature request User submitted feature request label Feb 15, 2023
@GoOz
Copy link
Owner

GoOz commented Feb 15, 2023

Ouais alors peut-être mais pas tout de suite. C'est un brin compliqué. Soit je change de DB en passant de localStorage à genre IndexedDB qui offrirait ce genre de souplesse ou bien je garde localStorage mais je préfixe toutes les données de perso supplémentaire. Ce qui fonctionnerait sûrement mais serait un peu plus chaotique.
Dans les deux cas y aurait un gros travail que je suis pas prêt à faire pour le moment.

Je garde malgré tout le ticket pour le futur. ;)

PS : Comment fait-on pour tagger directement une issue "Feature request" ?

C'est moi qui fait.

@rdugau
Copy link
Author

rdugau commented Feb 15, 2023

Tkt je savais que c'était une grosse feature.
Mais tu peux être sûr que ca va intéresser Seb

Quand tu dis "préfixer" le localStorage, y a pas une notion de folder ?
Tu gardes les mêmes noms de variables, avec un folder par pj.
Les variables ne sont plus "liées" directement au localstorage mais à une table.
Quand tu switches de pj, tu sauves la table en cours dans le folder du pj en cours. Et tu charges depuis le LS la table du pj sélectionné depuis le folder du PJ sélectionné.

Putain, c'est super mal expliqué. En bref
Un PJ=un folder=une table de param

En C/python ça se fait bien ce truc, en jscript je ne sais pas

@GoOz
Copy link
Owner

GoOz commented Feb 15, 2023

LocalStorage = juste une liste de clé/valeur par domaine (d'où l'idée de préfixer les clés pour savoir à quel perso les données appartiennent);
IndexedDB = on peut créer une DB (en clé/valeur aussi) mais surtout on peut créer plusieurs DB sur un même domaine ce qui serait en effect pratique pour le cas de multiple perso.

Le truc c'est que IndexdeDB est un peu plus compliqué à utiliser et instancier que LocalStorage qui lui ne nécessite rien de base.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature request User submitted feature request
Projects
None yet
Development

No branches or pull requests

2 participants