Nouvelle version de la librairie #1

Closed
opened 2 years ago by Pierretux · 8 comments
Collaborator

Bonjour,

Est ce que l'on peut mettre à jour la librairie js dans le plugins ?
Si oui, on fait une branche ou autre avant ?

Merci

Bonjour, Est ce que l'on peut mettre à jour la librairie js dans le plugins ? Si oui, on fait une branche ou autre avant ? Merci

Je suppose que ça dépend justement de quelle màj dans la lib fournie, si c'est des gros changements majeurs à priori oui, mettre le master dans une branche dédiée et changer la lib dans le nouveau master… mais sinon, juste bien vérifier qu'il y a un tag de la version actuelle, et ensuite modifier ? (possiblement avec une PR depuis une branche dev/nouvelle_lib_X_Y)

Je suppose que ça dépend justement de quelle màj dans la lib fournie, si c'est des gros changements majeurs à priori oui, mettre le master dans une branche dédiée et changer la lib dans le nouveau master… mais sinon, juste bien vérifier qu'il y a un tag de la version actuelle, et ensuite modifier ? (possiblement avec une PR depuis une branche dev/nouvelle_lib_X_Y)
Poster
Collaborator

Le tag existant est sur les derniers commit oui.
je ne sais pas faire de PR par contre.

Le tag existant est sur les derniers commit oui. je ne sais pas faire de PR par contre.
Collaborator
Pour faire une PR : https://contrib.spip.net/Equivalences-des-commandes-SVN-GIT#Proposer-des-modifications-via-pull-request-PR
Collaborator

Attention de bien vérifier la compatibilité, et que ça fonctionne toujours bien avec la compression des JS surtout, j'ai eu des problèmes avec ça et j'ai dû rétrograder sur une version précédente sur un site.

Attention de bien vérifier la compatibilité, et que ça fonctionne toujours bien avec la compression des JS surtout, j'ai eu des problèmes avec ça et j'ai dû rétrograder sur une version précédente sur un site.
Poster
Collaborator

@nicod j'ai tester le dernier zip et pas de problème pour le moment.
PR faites et à valider du coup

@nicod j'ai tester le dernier zip et pas de problème pour le moment. PR faites et à valider du coup
Collaborator

@pierretux : merci pour ta PR.
Il me semble toutefois que la révision "double quotte pour que le js fonctionne" n'est pas pertinente. Là où il n'y a pas de double quotes, c'est parce qu'on attend des booléens. Normal donc.

Comment as-tu constaté ce bug ?

@pierretux : merci pour ta PR. Il me semble toutefois que la révision "double quotte pour que le js fonctionne" n'est pas pertinente. Là où il n'y a pas de double quotes, c'est parce qu'on attend des booléens. Normal donc. Comment as-tu constaté ce bug ?
Poster
Collaborator

@peetdu le js est bloquer si tu ne valides pas la configuration de base, où alors il faut mettre des valeurs par défaut sinon ça bloque

@peetdu le js est bloquer si tu ne valides pas la configuration de base, où alors il faut mettre des valeurs par défaut sinon ça bloque
Collaborator

vérifier que ça fonctionne toujours bien avec la compression des JS
@nicod : tu es sûr que c'est Tarteaucitron qui est en cause ? Voir https://git.spip.net/spip-contrib-extensions/tarteaucitron/src/branch/master/tarteaucitron_pipelines.php#L31

> vérifier que ça fonctionne toujours bien avec la compression des JS @nicod : tu es sûr que c'est Tarteaucitron qui est en cause ? Voir https://git.spip.net/spip-contrib-extensions/tarteaucitron/src/branch/master/tarteaucitron_pipelines.php#L31
peetdu closed this issue 2 years ago
Sign in to join this conversation.
No Milestone
No Assignees
5 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.