Projet

Général

Profil

Development #19649

Ajouter le nom de l'app à laquelle on s'associe dans le Context de la page d'association

Ajouté par Josué Kouka il y a plus de 6 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Josué Kouka
Catégorie:
-
Version cible:
-
Début:
25 octobre 2017
Echéance:
% réalisé:

100%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

L'interet, sur la page d'association, est d' informer l'utilisateur sur le site avec lequel il va s'associer.
Ce nom peut etre surchargé par un paramètre dans les settings.


Fichiers

Révisions associées

Révision def3bf44 (diff)
Ajouté par Josué Kouka il y a plus de 6 ans

add application name and slug in association context (#19649)

Historique

#1

Mis à jour par Josué Kouka il y a plus de 6 ans

  • Sujet changé de Ajouter le nom de l'app à laquelle on s'associe dans le Context de la barre à Ajouter le nom de l'app à laquelle on s'associe dans le Context de la page d'association
#2

Mis à jour par Serghei Mihai il y a plus de 6 ans

Je rajouterai ici le slug de l'application en question, qui pourrait servir de nom de classe CSS à poser sur le body ou autre element pour personnaliser l'affichage.

#4

Mis à jour par Serghei Mihai il y a plus de 6 ans

Pour plus de clarté je poserais dans le contexte {'app': {'name': app_settings.name, 'slug': app_settings.slug}} pour pouvoir appeler dans le template {{ app.name }} et {{ app.slug }}
Dans le test, après le changement de settings.APP_NAME il faudrait vérifier aussi le slug.

#5

Mis à jour par Josué Kouka il y a plus de 6 ans

Serghei Mihai a écrit :

Pour plus de clarté je poserais dans le contexte {'app': {'name': app_settings.name, 'slug': app_settings.slug}} pour pouvoir appeler dans le template {{ app.name }} et {{ app.slug }}
Dans le test, après le changement de settings.APP_NAME il faudrait vérifier aussi le slug.

Ok, pris en compte dans ce patch.

#6

Mis à jour par Serghei Mihai il y a plus de 6 ans

Ton test doit échouer :)

#7

Mis à jour par Josué Kouka il y a plus de 6 ans

Yep, le slug devrait suivre le name

#8

Mis à jour par Josué Kouka il y a plus de 6 ans

  • Description mis à jour (diff)
#9

Mis à jour par Serghei Mihai il y a plus de 6 ans

Après avoir dormi un peu dessus je me dis que ton patch dans le commentaire #5 est bon.
Le slug ne doit pas bouger en fonction du nom de l'application.

On devrait avoir genre:

  • pour Meyzieu: slug=teamnet, name=Meyzieu
  • pour Rochefort: slug=teamnet, name=Rochefort

Donc ack pour le patch du commentaire #5.

#10

Mis à jour par Josué Kouka il y a plus de 6 ans

  • Statut changé de En cours à Résolu (à déployer)
  • % réalisé changé de 0 à 100
Author: Josue Kouka <jkouka@entrouvert.com>
Date:   Thu Nov 16 14:02:19 2017 +0100

    add application name and slug in association context (#19649)

#11

Mis à jour par Frédéric Péters il y a plus de 5 ans

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

Formats disponibles : Atom PDF