Projet

Général

Profil

Development #11401

Ajouter les frontends de création de compte au template.

Ajouté par Mikaël Ates il y a presque 8 ans. Mis à jour il y a presque 8 ans.

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

100%

Temps estimé:
Patch proposed:
Oui
Planning:

Fichiers

Révisions associées

Révision 1b0a2284 (diff)
Ajouté par Mikaël Ates il y a presque 8 ans

Add registration frontends to the registration template (fixes #11401).

Historique

#1

Mis à jour par Mikaël Ates il y a presque 8 ans

  • Fichier 0001-Ajout-des-frontends-de-cr-ation-de-compte-au-templat.patch ajouté
  • Patch proposed changé de Non à Oui
#2

Mis à jour par Mikaël Ates il y a presque 8 ans

Nécessite authentic2 >= v2.1.20-1009.

#3

Mis à jour par Frédéric Péters il y a presque 8 ans

  • Projet changé de Publik à Intégrations graphiques Publik
#4

Mis à jour par Frédéric Péters il y a presque 8 ans

Nécessite authentic2 >= v2.1.20-1009.

À renseigner dans le debian/control. Avec un point et non un tiret, dans le numéro de version.

Mais est-ce vrai ? Il me esmble que {% for id, block in frontends.items %} ne poserait pas de problème sur un déploiement actuel.

Aussi, tant qu'à avoir "id", autant l'utiliser pour marquer le <div> de manière unique.

#5

Mis à jour par Mikaël Ates il y a presque 8 ans

  • Fichier 0001-Ajout-des-frontends-de-cr-ation-de-compte-au-templat.patch ajouté

Cela ne pose pas de problème si authentic2 n'est pas >= v2.1.20-1009 mais le patch n'est pas effectif.

#6

Mis à jour par Mikaël Ates il y a presque 8 ans

  • Fichier 0001-Ajout-des-frontends-de-cr-ation-de-compte-au-templat.patch supprimé
#7

Mis à jour par Frédéric Péters il y a presque 8 ans

Étrange je pensais avoir fait une série de remarques et la seule trace qui en reste c'est le "À renseigner dans le debian/control...".

Dans mon souvenir, je notais que le message de commit devait être en anglais et qu'idéalement le patch ne changerait pas l'indentation du fichier, ce qui lui permettrait d'être bien plus clair dans ce qu'il change.

L'id juste comme ça, c'est peut-être un peu court, id="registration-{{id}}" plutôt ?

#8

Mis à jour par Mikaël Ates il y a presque 8 ans

  • Fichier 0001-Add-registration-frontends-to-the-registration-templ.patch ajouté

"À renseigner dans le debian/control...".

Pas de dépendance donc.

commit devait être en anglais

Done.

le patch ne changerait pas l'indentation du fichier

Done.

L'id juste comme ça, c'est peut-être un peu court, id="registration-{{id}}" plutôt ?

J'ai opté pour frontend-registration-{{id}}.

#9

Mis à jour par Mikaël Ates il y a presque 8 ans

  • Fichier 0001-Ajout-des-frontends-de-cr-ation-de-compte-au-templat.patch supprimé
#10

Mis à jour par Mikaël Ates il y a presque 8 ans

  • Fichier 0001-Add-registration-frontends-to-the-registration-templ.patch ajouté

J'ai opté pour frontend-registration-{{id}}.

Et donc aussi ajout d'un id pour le block email.

#11

Mis à jour par Mikaël Ates il y a presque 8 ans

  • Fichier 0001-Add-registration-frontends-to-the-registration-templ.patch supprimé
#13

Mis à jour par Mikaël Ates il y a presque 8 ans

  • Fichier 0001-Add-registration-frontends-to-the-registration-templ.patch supprimé
#14

Mis à jour par Frédéric Péters il y a presque 8 ans

ok.

#15

Mis à jour par Mikaël Ates il y a presque 8 ans

  • Statut changé de Nouveau à Résolu (à déployer)
  • % réalisé changé de 0 à 100
#16

Mis à jour par Mikaël Ates il y a presque 8 ans

Appliqué par commit publik:commit:1b0a2284b4d9e999c198b65639551caf5ee90286.

#17

Mis à jour par Frédéric Péters il y a presque 8 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF