Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix #3609 : mise a jour de la page de contact #3613

Merged
merged 3 commits into from Jun 17, 2016
Merged

Fix #3609 : mise a jour de la page de contact #3613

merged 3 commits into from Jun 17, 2016

Conversation

gustavi
Copy link
Contributor

@gustavi gustavi commented May 14, 2016

Q R
Type de modification évolution
Ticket(s) (issue(s)) concerné(s) #3609

Explications

cc @cgabard @SpaceFox @GerardPaligot

J'ai réécrit entièrement la page pour qu'on utilise des objets et qu'on puisse modifier cette page sans pour autant faire une PR.

QA

Ne pas merger avant #3611 car il y aurar du conflit et je dois rebase celle-ci. La QA peut toutefois être faite.

  • Lancer les migrations avec python manage.py migrate
  • Ajouter un ou plusieurs groupes via l'admin Django
  • Aller sur la page de contact et vérifier son bon fonctionnement

@Situphen
Copy link
Member

Situphen commented May 15, 2016

C'est possible que tu internationalises les quelques chaînes de caractères de models.py ? :)

Par ailleurs, il y a deux fois le champ Email : dans ton paste.

@gustavi
Copy link
Contributor Author

gustavi commented May 16, 2016

Mis à jour.

@coveralls
Copy link

coveralls commented May 16, 2016

Coverage Status

Coverage increased (+0.01%) to 87.595% when pulling 8c32abf on gustavi:fix-3609 into 195a586 on zestedesavoir:dev.

{% endif %}
{% with users=contact_group.group.user_set.all %}
{% if users %}
<p>Les membres sont les suivants :</p>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Un petit {% trans "" %} ? :)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Décidément j'étais fatigué en codant ça !

@coveralls
Copy link

coveralls commented May 17, 2016

Coverage Status

Coverage increased (+0.01%) to 87.595% when pulling 216b100 on gustavi:fix-3609 into 195a586 on zestedesavoir:dev.

@gustavi
Copy link
Contributor Author

gustavi commented May 17, 2016

Mis à jour.


**À faire après la mise en production par une personne ayant les droits admin**.

Il faut ajouter les différents groupes de la page de contact via l'administration Django. Les informations pour les créer sont sur http://paste.awesom.eu/B598.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pourquoi un lien externe ? Met toutes les informations dans ce fichier ci.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Un peu lourd non ?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 @GerardPaligot

Ce n'est pas si lourd que ça une vingtaine de ligne et évite de passer par un site externe (pas besoin de quitter le shell & on est sûr de garder l'historique)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pas mieux que @Situphen

@gustavi
Copy link
Contributor Author

gustavi commented May 26, 2016

Mis à jour.

@coveralls
Copy link

coveralls commented May 26, 2016

Coverage Status

Coverage increased (+0.01%) to 87.595% when pulling aa8c820 on gustavi:fix-3609 into 5768c4e on zestedesavoir:dev.

@Emeric54
Copy link
Contributor

Emeric54 commented Jun 6, 2016

QA => 🆗

@GerardPaligot
Copy link
Member

@gustavi need rebase

@gustavi
Copy link
Contributor Author

gustavi commented Jun 17, 2016

rebased !

@coveralls
Copy link

coveralls commented Jun 17, 2016

Coverage Status

Coverage increased (+0.09%) to 53.073% when pulling 3a9d49e on gustavi:fix-3609 into a902374 on zestedesavoir:dev.

@coveralls
Copy link

coveralls commented Jun 17, 2016

Coverage Status

Coverage increased (+34.4%) to 87.42% when pulling 3a9d49e on gustavi:fix-3609 into a902374 on zestedesavoir:dev.

@GerardPaligot GerardPaligot merged commit 0040f30 into zestedesavoir:dev Jun 17, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants