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

Adapter l'affichage du forum pour les membres en lecture seule #5684

Closed
Arnaud-D opened this issue Apr 7, 2020 · 1 comment · Fixed by #6234
Closed

Adapter l'affichage du forum pour les membres en lecture seule #5684

Arnaud-D opened this issue Apr 7, 2020 · 1 comment · Fixed by #6234
Assignees
@SpaceFox @jeanlapostolle
Labels
C-Front Concerne l'interface du site Facile Bon ticket pour débuter pour rejoindre le développement ! S-Évolution Ajoute de nouvelles fonctionnalités

Comments

@Arnaud-D
Copy link
Contributor

Arnaud-D commented Apr 7, 2020 •
edited
Loading

Suite à une discussion sur le forum.

Comportement observé : en tant que membre en lecture, je peux remplir le formulaire de nouveau message, mais reçoit une 403 quand j'essaie de l'envoyer.

Comportement souhaité : en tant que membre en lecture seule, je ne vois pas le formulaire, mais vois un message m'indiquant que je ne peux pas poster, car je suis en lecture seule. La 403 existe toujours, mais il faut faire exprès pour la voir.

Pour résoudre le problème, ça se passe dans le template : templates/misc/message_form.html et on peut rajouter une condition avec not user.can_write_now.

Il faut aussi quelque chose de similaire pour les nouveaux sujets. Ça se passe ici avec une solution similaire :

  • /templates/forum/topic/index.html
  • templates/forum/category/forum.html
@Arnaud-D Arnaud-D added C-Back Concerne le back-end Django C-Front Concerne l'interface du site S-Évolution Ajoute de nouvelles fonctionnalités labels Apr 7, 2020
@Situphen Situphen changed the title Adapter l'affichage du forum pour les membres en lecture seules Adapter l'affichage du forum pour les membres en lecture seule Apr 7, 2020
@Arnaud-D Arnaud-D added Facile Bon ticket pour débuter pour rejoindre le développement ! and removed C-Back Concerne le back-end Django labels Apr 15, 2020
@jeanlapostolle
Copy link
Contributor

Je prends

@jeanlapostolle jeanlapostolle mentioned this issue Jun 23, 2020
Closed
@AmauryCarrade AmauryCarrade linked a pull request Jun 23, 2020 that will close this issue
fix #5684: Adapte l'affichage du forum pour les membres en lecture seule #5837
Closed
SpaceFox pushed a commit to SpaceFox/zds-site that referenced this issue Jan 23, 2022
@jeanlapostolle @SpaceFox
fix zestedesavoir#5684
ce7c0d1
SpaceFox added a commit to SpaceFox/zds-site that referenced this issue Jan 23, 2022
@SpaceFox
Affichage en LS : utilisation du bon objet et tests (zestedesavoir#5684)
11ad4e0
@SpaceFox SpaceFox mentioned this issue Jan 23, 2022
Merged
@Arnaud-D Arnaud-D linked a pull request Jan 23, 2022 that will close this issue
Affichage des membres en LS (reprise de la PR #5837) #6234
Merged
@SpaceFox SpaceFox self-assigned this Jan 24, 2022
SpaceFox added a commit to SpaceFox/zds-site that referenced this issue Jan 24, 2022
@SpaceFox
Affichage en LS : utilisation du bon objet et tests (zestedesavoir#5684)
7640fd4
Situphen pushed a commit that referenced this issue Jan 31, 2022
@jeanlapostolle @Situphen
fix #5684
854f1ec
Situphen pushed a commit that referenced this issue Jan 31, 2022
@SpaceFox @Situphen
Affichage en LS : utilisation du bon objet et tests (#5684)
a91d572
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@SpaceFox SpaceFox

@jeanlapostolle jeanlapostolle

Labels
C-Front Concerne l'interface du site Facile Bon ticket pour débuter pour rejoindre le développement ! S-Évolution Ajoute de nouvelles fonctionnalités
Projects
Triage des tickets pour débutant
Archived in project
Milestone
No milestone
3 participants
@SpaceFox @jeanlapostolle @Arnaud-D