Projet

Général

Profil

Development #82406

Message du popup de confirmation d'action manuelle

Ajouté par Cédric Dourdent il y a 7 mois. Mis à jour il y a 6 mois.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
16 octobre 2023
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

Il me semble qu''il y a peu, la popup de confirmation d'action manuelle affichait "Etes-vous sur.e?" ou quelque chose de plus séduisant que ce qu'il y a maintenant "true".

Voir PJ

On peut faire quelque chose de ça ?
(je sais que la question va tomber demain chez Parsifal :/)


Fichiers

Capture d’écran du 2023-10-16 14-14-21.png (16,3 ko) Capture d’écran du 2023-10-16 14-14-21.png popup confirmation action manuelle Cédric Dourdent, 16 octobre 2023 14:18

Demandes liées

Lié à w.c.s. - Development #82066: paramètre pour configurer le texte de confirmation de l'action de saut manuelFermé06 octobre 2023

Actions
Lié à w.c.s. - Development #82585: Texte par défaut dans la boîte de confirmation si aucun texte personnalisé n'est défini, actuellement « true »Fermé19 octobre 2023

Actions
Lié à w.c.s. - Development #82660: Message de confirmation de saut manuel : Ajouter un message par défautFermé20 octobre 2023

Actions

Révisions associées

Révision 74435152 (diff)
Ajouté par Lauréline Guérin il y a 7 mois

workflows: fix default confirmation in choice action (#82406)

Historique

#2

Mis à jour par Lauréline Guérin il y a 7 mois

dû à #82066

je pense qu'ici:

  function prepare_confirmation_buttons() {
    $('button[data-ask-for-confirmation]').off('click').on('click', function() {
      var text = $(this).data('ask-for-confirmation');
      if (text === 'true') {
        text = WCS_I18N.confirmation;
      }
      if (confirm(text) != true) {
        return false;
      } else {
        return true;
      }
    });
  }

il faudrait écrire if (text === true)

#3

Mis à jour par Lauréline Guérin il y a 7 mois

  • Lié à Development #82066: paramètre pour configurer le texte de confirmation de l'action de saut manuel ajouté
#4

Mis à jour par Corentin Séchet il y a 7 mois

https://stackoverflow.com/questions/9921076/retrieve-boolean-data-from-data-attribute-in-jquery

C'est jQuery qui caste la valeur "true" en booléen en javascript : il faut effectivement écrire if(text === true).

#5

Mis à jour par Robot Gitea il y a 7 mois

  • Tracker changé de Autre à Development
  • Statut changé de Nouveau à Solution proposée
  • Assigné à mis à Lauréline Guérin

Lauréline Guérin (lguerin) a ouvert une pull request sur Gitea concernant cette demande :

#6

Mis à jour par Robot Gitea il y a 7 mois

  • Statut changé de Solution proposée à Solution validée

Corentin Sechet (csechet) a approuvé une pull request sur Gitea concernant cette demande :

#7

Mis à jour par Robot Gitea il y a 7 mois

  • Statut changé de Solution validée à Résolu (à déployer)

Lauréline Guérin (lguerin) a mergé une pull request sur Gitea concernant cette demande :

#8

Mis à jour par Lauréline Guérin il y a 6 mois

  • Lié à Development #82585: Texte par défaut dans la boîte de confirmation si aucun texte personnalisé n'est défini, actuellement « true » ajouté
#9

Mis à jour par Lauréline Guérin il y a 6 mois

  • Lié à Development #82660: Message de confirmation de saut manuel : Ajouter un message par défaut ajouté
#10

Mis à jour par Transition automatique il y a 6 mois

  • Statut changé de Résolu (à déployer) à Solution déployée
#11

Mis à jour par Transition automatique il y a 4 mois

Automatic expiration

Formats disponibles : Atom PDF