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.

Sign up for GitHub

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

Jump to bottom

Ajoute Discord dans le pied de page #6514

Merged
merged 3 commits into from
Sep 3, 2023
Merged

Ajoute Discord dans le pied de page #6514

merged 3 commits into from
Sep 3, 2023

Conversation

AmauryCarrade
Copy link
Member

@AmauryCarrade AmauryCarrade commented Jun 26, 2023 •
edited by Arnaud-D
Loading

Ajoute le lien vers le serveur Discord dans le pied de page, suite à la décision de l'AGO du 24 juin 2023 de rendre e serveur Discord officiel.

Cette PR change uniquement la configuration car le travail a déjà été fait dans #5874. Cependant, depuis 2020, le logo de Discord a changé ; nous devrions mettre le logo à jour avant de fusionner.

Contrôle qualité (inutile pour le moment)

La QA a déjà été faite dans #5874 mais si on veut quand même vérifier :

  1. Lancer le site avec make run ou make run-fast.
  2. Vérifier que le logo Discord est présent dans le pied de page.
  3. Vérifier que le lien d'invitation pointe bien vers le serveur de Zeste de Savoir.
  4. Vérifier que le logo de Discord est bien le nouveau logo.

@AmauryCarrade AmauryCarrade added the C-Front Concerne l'interface du site label Jun 26, 2023
@coveralls
Copy link

coveralls commented Jun 26, 2023 •
edited
Loading

Coverage Status

coverage: 88.393%. remained the same when pulling 798c30b on discord-is-now-official into b6fd744 on dev.

@philippemilink
Copy link
Member

D'où vient l'image qui avait été ajoutée dans #5884 ? @helmasaur ?

@Arnaud-D
Copy link
Contributor

Le branding actuel est dispo ici : https://discord.com/branding

@philippemilink
Copy link
Member

Vu en réunion de dev's : pas la peine d'attendre de changer le logo pour fusionner. Donc une QA de l'état actuel de la PR est suffisant.

@AmauryCarrade
Copy link
Member Author

N'est-ce pas contraire aux règles d'usage de l'identité de Discord ? Puis ça ferait obsolète…

@Arnaud-D Arnaud-D marked this pull request as ready for review August 26, 2023 14:40
@Arnaud-D
Copy link
Contributor

Arnaud-D commented Aug 26, 2023 •
edited
Loading

J'ai intégré le nouveau logo Discord et ses variantes. Au passage, Discord a changé sa route pour les liens d'invitation et j'ai pris la nouvelle.

AmauryCarrade and others added 3 commits September 3, 2023 13:19
@AmauryCarrade @Arnaud-D
Adds Discord Server to the footer
490bed1 
Updates the app config to display the discord server invite link into the footer.
@Arnaud-D
Modifie l'URL d'invitation
c70d40c 
Discord a changé sa route, celle-ci est la nouvelle.
@Arnaud-D
Met à jour le logo Discord
6d0c464
philippemilink
philippemilink approved these changes Sep 3, 2023
View reviewed changes
Copy link
Member

@philippemilink philippemilink left a comment

Choose a reason for hiding this comment

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

QA OK ✔️

@philippemilink philippemilink merged commit 7297871 into dev Sep 3, 2023
16 checks passed
@philippemilink philippemilink deleted the discord-is-now-official branch September 3, 2023 13:36
Migwel pushed a commit to Migwel/zds-site that referenced this pull request Sep 12, 2023
@AmauryCarrade @Arnaud-D
Ajoute Discord dans le pied de page (zestedesavoir#6514)
d4c2cec 
* Adds Discord Server to the footer

Updates the app config to display the discord server invite link into the footer.

* Modifie l'URL d'invitation

Discord a changé sa route, celle-ci est la nouvelle.

* Met à jour le logo Discord

---------

Co-authored-by: Arnaud-D <[email protected]>
@AmauryCarrade
Copy link
Member Author

AmauryCarrade commented Sep 13, 2023 •
edited
Loading

J'ai intégré le nouveau logo Discord et ses variantes. Au passage, Discord a changé sa route pour les lien d'invitation et j'ai pris la nouvelle.

Pour info, les liens en .gg ont toujours redirigé vers discord(app).com/invite, c'est un raccourci officiel et pas du tout déprécié :) (Mais ça ne change rien)

Merci pour les logos ! Je n'avais pas vu sur le moment, d'où le délai…

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@philippemilink philippemilink philippemilink approved these changes

Assignees
No one assigned
Labels
C-Front Concerne l'interface du site
Projects
Suivi des PR
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@AmauryCarrade @coveralls @philippemilink @Arnaud-D