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

le choix d'un thème dans un nouveau lodex s'impose dans les autres lodex d'une instance-globale #2256

Open
HeleneCreusot opened this issue Dec 9, 2024 · 2 comments
Assignees
Labels
⚠️ Minor defect Le problème ne permet pas d'optimiser le processus standard ou concerne un cas d'usage spécifique 👷‍♂️ Ready to go Le besoin ou le problème est validé, il prêt à être développé.. 👩‍🎨 UX Design L'affichage ou l’ergonomie pourraient être améliorées

Comments

@HeleneCreusot
Copy link
Collaborator

HeleneCreusot commented Dec 9, 2024

Décrire le bogue
dans une instance globale, le theme sélectionné dans un nouveau lodex s'impose aux autres lodex. Du coup, il faut aller dans chaque config de chaque lodex et revalider le theme choisi.

Reproduire

  1. dans une nouvelle instance globale, créer un lodex en plus du default.
  2. dans la configuration de ce nouveau lodex, sélectionner un thème autre que 'syteme', enregistrer.
  3. dans l'insance globale, cliquer sur default connecter user et voir que le lodex default n'affiche pas le theme système comme indiqué dans sa config, mais le thème du précédent lodex que l'on vient de configurer

Comportement attendu
les themes doivent être liés aux lodex d'une instance globale

Environnement (veuillez compléter les informations suivantes) :
tests réalisés avec lodex 14.0.67

@HeleneCreusot HeleneCreusot added the Triage Nouveau problème ou besoin à étudier et à classer label Dec 9, 2024
@HeleneCreusot HeleneCreusot changed the title l'ajout d'un thème dans un nouveau lodex s'impose dans les autres lodex d'une instance l'ajout d'un thème dans un nouveau lodex s'impose dans les autres lodex d'une instance-globale Dec 9, 2024
@HeleneCreusot HeleneCreusot changed the title l'ajout d'un thème dans un nouveau lodex s'impose dans les autres lodex d'une instance-globale le choix d'un thème dans un nouveau lodex s'impose dans les autres lodex d'une instance-globale Dec 9, 2024
@touv
Copy link
Contributor

touv commented Dec 9, 2024

Le lien vers les fichiers css n'est pas bon

Voici quelques étapes

  1. appel templae ejs

    return renderTemplate(lodexTheme.index, extendedData);

  2. appel render du front
    https://github.com/Inist-CNRS/lodex/blob/master/src/api/controller/front.js#L188-L195

  3. choix de la config (et donc du theme)

    const configTenant = await ctx.configTenantCollection.findLast();

@yannguillemin yannguillemin added ⚠️ Minor defect Le problème ne permet pas d'optimiser le processus standard ou concerne un cas d'usage spécifique 👩‍🎨 UX Design L'affichage ou l’ergonomie pourraient être améliorées 👷‍♂️ Ready to go Le besoin ou le problème est validé, il prêt à être développé.. and removed Triage Nouveau problème ou besoin à étudier et à classer labels Dec 9, 2024
@touv
Copy link
Contributor

touv commented Dec 13, 2024

j'ai vu également ce problème, par contre je n'arrive pas encore à le reproduire dans un environnement de développement...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚠️ Minor defect Le problème ne permet pas d'optimiser le processus standard ou concerne un cas d'usage spécifique 👷‍♂️ Ready to go Le besoin ou le problème est validé, il prêt à être développé.. 👩‍🎨 UX Design L'affichage ou l’ergonomie pourraient être améliorées
Projects
None yet
Development

When branches are created from issues, their pull requests are automatically linked.

4 participants