Rajout dans la configuration du plugins la possibilité de pouvoir surcharger un premier texte "alertBigPrivacy" #5

Merged
peetdu merged 4 commits from dev/surcharge_lang into master 1 year ago
Collaborator

Bonjour,

Demande qui rejoint #3

Bonjour, Demande qui rejoint #3
Pierretux added 1 commit 2 years ago
Pierretux added 1 commit 2 years ago
Pierretux added 1 commit 2 years ago
Poster
Collaborator

@Akilia on peut voir pour cet ajout ? Merci

@Akilia on peut voir pour cet ajout ? Merci
Collaborator

Pourquoi pas, mais je trouve que c'est quand même très spécifique.
Je me demande si cela vaudrait pas plutôt le coup d'ajouter dans la doc la méthode pour surcharger des chaînes de langue du JS.

De plus, as-tu tester sur un site multingue ?

Pourquoi pas, mais je trouve que c'est quand même très spécifique. Je me demande si cela vaudrait pas plutôt le coup d'ajouter dans la doc la méthode pour surcharger des chaînes de langue du JS. De plus, as-tu tester sur un site multingue ?
Poster
Collaborator

Non je ne gère pas de multilingue.
Mais ok pour de la doc JS dans cas, je te laisse faire car je vois pas pour le multilingue pour le coup.

Non je ne gère pas de multilingue. Mais ok pour de la doc JS dans cas, je te laisse faire car je vois pas pour le multilingue pour le coup.
peetdu closed this pull request 1 year ago
Collaborator

Hello

Suis dans un cas d'usage qui est multilingue.
De plus le texte étant propre à la gestion du site et non du webmestre, il me semble pertinent qu'on puisse gérer le texte depuis le plugin.

Les accès aux chaine de langues sont d'un niveau technique alors que le texte d'information est d'un niveau éditorial (amha)

Hello Suis dans un cas d'usage qui est multilingue. De plus le texte étant propre à la gestion du site et non du webmestre, il me semble pertinent qu'on puisse gérer le texte depuis le plugin. Les accès aux chaine de langues sont d'un niveau technique alors que le texte d'information est d'un niveau éditorial (amha)
Owner

Je suis assez d'accord avec @cam.lafit :)

Il s'agit plutôt de texte éditoriaux il me semble aussi, donc par défaut ya un texte prévu ok, mais si on veut les changer, c'est plutôt à des admins dans l'interface, et non pas à des personnes techniques.

Du coup on pourrait rouvrir cette PR.

Par contre faut penser à passer |propre sur les textes, pour pouvoir utiliser les raccourcis mais surtout les multis.

Je suis assez d'accord avec @cam.lafit :) Il s'agit plutôt de texte éditoriaux il me semble aussi, donc par défaut ya un texte prévu ok, mais si on veut les changer, c'est plutôt à des admins dans l'interface, et non pas à des personnes techniques. Du coup on pourrait rouvrir cette PR. Par contre faut penser à passer |propre sur les textes, pour pouvoir utiliser les raccourcis mais surtout les multis.
cam.lafit reopened this pull request 1 year ago
cam.lafit added 11 commits 1 year ago
e57c59fe7f Utiliser l'objet de l'environnement :
cb459f3e37 Supporter les champs multi
cam.lafit force-pushed dev/surcharge_lang from cb459f3e37 to 066721a27a 1 year ago
cam.lafit added 1 commit 1 year ago
a6e407dff5 Supporter les champs multi
cam.lafit force-pushed dev/surcharge_lang from a6e407dff5 to 0be5ff36f2 1 year ago
peetdu merged commit 54b85e0e03 into master 1 year ago
The pull request has been merged as 54b85e0e03.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
4 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.