afficher un encart pour signaler les nouvelles versions #5691

Open
b_b wants to merge 2 commits from issue_5690 into master
b_b commented 2 months ago
Owner

dans toutes les pages de l'espace privé pour les webmestres

Fix: #5690

Reste la question de l'item de langue pour le bouton "Lancer la mise à jour", on a bien un item "Mettre à jour", mais il est dans SVP.

dans toutes les pages de l'espace privé pour les webmestres Fix: #5690 Reste la question de l'item de langue pour le bouton "Lancer la mise à jour", on a bien un item "Mettre à jour", mais il est dans SVP.
b_b commented 2 months ago
Poster
Owner

Seul problème, le pipeline affiche_milieu s'insère dans le bas de la page des plugins (et non en haut comme d'habitude).

Seul problème, le pipeline affiche_milieu s'insère dans le bas de la page des plugins (et non en haut comme d'habitude).
maieul reviewed 2 months ago
@ -0,0 +1,5 @@
[(#BOITE_OUVRIR{'',notice})]
<p><:nouvelle_version_spip{version=#CONFIG{derniere_maj_notifiee}}:></p>
#BOITE_PIED
<a class="btn" href="#URL_SITE_SPIP/spip_loader.php">Lancer la mise à jour</a>
Collaborator

[(#CHEMIN{spip_loader.php}|oui)<a class="btn" href="#URL_SITE_SPIP/spip_loader.php">Lancer la mise à jour</a>]

plutot non `?

`[(#CHEMIN{spip_loader.php}|oui)<a class="btn" href="#URL_SITE_SPIP/spip_loader.php">Lancer la mise à jour</a>]` plutot non `?
b_b commented 2 months ago
Poster
Owner

Oui c'était en cours ;)

Oui c'était en cours ;)
b_b marked this conversation as resolved
Collaborator

Seul problème, le pipeline affichage_final s'insère dans le bas de la page des plugins (et non en haut comme d'habitude).

afficher_milieu tu veux dire ?

> Seul problème, le pipeline affichage_final s'insère dans le bas de la page des plugins (et non en haut comme d'habitude). `afficher_milieu` tu veux dire ?
b_b force-pushed issue_5690 from 89be746074 to 6107918142 2 months ago
b_b commented 2 months ago
Poster
Owner

afficher_milieu tu veux dire ?

Vi, typo corrigée :)

> `afficher_milieu` tu veux dire ? Vi, typo corrigée :)
Collaborator

Seul problème, le pipeline affiche_milieu s'insère dans le bas de la page des plugins (et non en haut comme d'habitude).

spip/svp#4896 corrige lorsqu'on a SVP.

Il faut aussi une correction pour le core, je n'ose pas pusher pour l'instant, car je ne sais pas si tu bosse sur la branche (et peut être que je devrais faire une autre branche)

> Seul problème, le pipeline affiche_milieu s'insère dans le bas de la page des plugins (et non en haut comme d'habitude). https://git.spip.net/spip/svp/pulls/4896 corrige lorsqu'on a SVP. Il faut aussi une correction pour le core, je n'ose pas pusher pour l'instant, car je ne sais pas si tu bosse sur la branche (et peut être que je devrais faire une autre branche)
b_b changed title from WIP: afficher un encart pour signaler les nouvelles versions to Afficher un encart pour signaler les nouvelles versions 2 months ago
b_b commented 2 months ago
Poster
Owner

À noter, cette PR dépend de #5658

À noter, cette PR dépend de https://git.spip.net/spip/spip/pulls/5658
b_b force-pushed issue_5690 from 6107918142 to 8949497fee 2 months ago
b_b changed title from Afficher un encart pour signaler les nouvelles versions to WIP: afficher un encart pour signaler les nouvelles versions 2 months ago
b_b commented 2 months ago
Poster
Owner

Concernant l'item de langue je propose :

diff --git a/ecrire/lang/ecrire_fr.php b/ecrire/lang/ecrire_fr.php
index 9ea42454e9..a3402660b7 100644
--- a/ecrire/lang/ecrire_fr.php
+++ b/ecrire/lang/ecrire_fr.php
@@ -53,6 +53,7 @@ return [
 	'bouton_envoyer_message' => 'Message définitif : envoyer',
 	'bouton_fermer' => 'Fermer',
 	'bouton_mettre_a_jour_base' => 'Mettre à jour la base de données',
+	'bouton_mettre_a_jour_spip' => 'Lancer la mise à jour',
 	'bouton_modifier' => 'Modifier',
 	'bouton_radio_afficher' => 'Afficher',
 	'bouton_radio_apparaitre_liste_redacteurs_connectes' => 'Apparaître dans la liste des rédacteurs connectés',
diff --git a/prive/squelettes/inclure/mise_a_jour.html b/prive/squelettes/inclure/mise_a_jour.html
index 72e28dee36..b2f8d1255a 100644
--- a/prive/squelettes/inclure/mise_a_jour.html
+++ b/prive/squelettes/inclure/mise_a_jour.html
@@ -1,5 +1,5 @@
 [(#BOITE_OUVRIR{'',notice})]
 	<p><:nouvelle_version_spip{version=#CONFIG{derniere_maj_notifiee}}:></p>
 	[#BOITE_PIED(#CHEMIN{spip_loader.php}|oui)
-	<a class="btn" href="#URL_SITE_SPIP/spip_loader.php">Lancer la mise à jour</a>]
+	<a class="btn" href="#URL_SITE_SPIP/spip_loader.php"><:bouton_mettre_a_jour_spip:></a>]
 [(#BOITE_FERMER)]
Concernant l'item de langue je propose : ```diff diff --git a/ecrire/lang/ecrire_fr.php b/ecrire/lang/ecrire_fr.php index 9ea42454e9..a3402660b7 100644 --- a/ecrire/lang/ecrire_fr.php +++ b/ecrire/lang/ecrire_fr.php @@ -53,6 +53,7 @@ return [ 'bouton_envoyer_message' => 'Message définitif : envoyer', 'bouton_fermer' => 'Fermer', 'bouton_mettre_a_jour_base' => 'Mettre à jour la base de données', + 'bouton_mettre_a_jour_spip' => 'Lancer la mise à jour', 'bouton_modifier' => 'Modifier', 'bouton_radio_afficher' => 'Afficher', 'bouton_radio_apparaitre_liste_redacteurs_connectes' => 'Apparaître dans la liste des rédacteurs connectés', diff --git a/prive/squelettes/inclure/mise_a_jour.html b/prive/squelettes/inclure/mise_a_jour.html index 72e28dee36..b2f8d1255a 100644 --- a/prive/squelettes/inclure/mise_a_jour.html +++ b/prive/squelettes/inclure/mise_a_jour.html @@ -1,5 +1,5 @@ [(#BOITE_OUVRIR{'',notice})] <p><:nouvelle_version_spip{version=#CONFIG{derniere_maj_notifiee}}:></p> [#BOITE_PIED(#CHEMIN{spip_loader.php}|oui) - <a class="btn" href="#URL_SITE_SPIP/spip_loader.php">Lancer la mise à jour</a>] + <a class="btn" href="#URL_SITE_SPIP/spip_loader.php"><:bouton_mettre_a_jour_spip:></a>] [(#BOITE_FERMER)] ```
Owner

Ah tu veux dire que c’est mis systématiquement sur toutes les pages de l’espace privé ?

Ah tu veux dire que c’est mis systématiquement sur **toutes** les pages de l’espace privé ?
Collaborator

Pourquoi pas simplement "Mettre à jour" ?

Pourquoi pas simplement "Mettre à jour" ?
b_b commented 2 months ago
Poster
Owner

Ah tu veux dire que c’est mis systématiquement sur toutes les pages de l’espace privé ?

Vi, au départ c'était uniquement sur l'accueil du privé, puis j'ai modifié en fonction de la spec de @rastapopoulos

> Ah tu veux dire que c’est mis systématiquement sur **toutes** les pages de l’espace privé ? Vi, au départ c'était uniquement sur l'accueil du privé, puis j'ai modifié en fonction de la spec de @rastapopoulos
Owner

Pourquoi pas simplement "Mettre à jour" ?

yep plus court plus simple plus efficace, pour un bouton, je trouve

> Pourquoi pas simplement "Mettre à jour" ? yep plus court plus simple plus efficace, pour un bouton, je trouve
b_b force-pushed issue_5690 from 8949497fee to 8b566572bd 2 months ago
b_b commented 2 months ago
Poster
Owner

Voilà, la PR est à jour avec la chaîne de langue.

Voilà, la PR est à jour avec la chaîne de langue.
b_b changed title from WIP: afficher un encart pour signaler les nouvelles versions to afficher un encart pour signaler les nouvelles versions 2 months ago
b_b force-pushed issue_5690 from 0ea8a5d305 to e4c282b305 2 months ago
b_b force-pushed issue_5690 from e4c282b305 to 4ac68e2651 2 months ago
Owner

C’est à merger cela au fait du coup ?

C’est à merger cela au fait du coup ?
Owner

Il manque un changelog ^^

Il manque un changelog ^^
b_b commented 2 weeks ago
Poster
Owner

Je n'ai pas envoyé de changelog car personne n'avait approuvé jusqu'ici :)

Je n'ai pas envoyé de changelog car personne n'avait approuvé jusqu'ici :)
marcimat force-pushed issue_5690 from 4ac68e2651 to 0c4754795e 1 week ago
marcimat reviewed 1 week ago
@ -0,0 +1,5 @@
[(#BOITE_OUVRIR{'',notice})]
<p><:nouvelle_version_spip{version=#CONFIG{derniere_maj_notifiee}}:></p>
Owner

En voulant tester (je n’avais pas la PR indiquant derniere_maj_notifiee dans un premier temps) mais néanmoins je me demande si c’est la bonne config à mettre ici.

Parce qu’après tout, on ne cherche pas à savoir ce qui a été notifié, mais bien la version disponible (même si ça doit être souvent la même)

En voulant tester (je n’avais pas la PR indiquant `derniere_maj_notifiee` dans un premier temps) mais néanmoins je me demande si c’est la bonne config à mettre ici. Parce qu’après tout, on ne cherche pas à savoir ce qui a été notifié, mais bien la version disponible (même si ça doit être souvent la même)
This pull request can be merged automatically.
You are not authorized to merge this pull request.
Sign in to join this conversation.
Loading…
There is no content yet.