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

Affichage des membres en LS (reprise de la PR #5837) #6234

Merged
merged 3 commits into from
Jan 31, 2022

Conversation

SpaceFox
Copy link
Contributor

Affiche un message d’avertissement à la place du formulaire si le membre est en LS. Supprime aussi les liens d’édition et de création de sujets.

Numéro du ticket concerné : #5684

Contrôle qualité

  • Avec un compte normal (droits d’écriture), vérifier que :
    • Le lien « Nouveau sujet » est présent sur les pages de forum et de sujet
    • Le lien « Éditer le sujet » est présent sur les sujets créés par l’utilisateur
    • Le formulaire de rédaction est bien présent
  • Avec un compte en lecture seule, vérifier que :
    • Le lien « Nouveau sujet » est absent des pages de forum et de sujets
    • Le lien « Éditer le sujet » est absent des sujets créés par l’utilisateur
    • Le formulaire de rédaction est remplacé par un avertissement qui indique que l’utilisateur est en LS.

@coveralls
Copy link

coveralls commented Jan 23, 2022

Coverage Status

Coverage increased (+0.006%) to 87.025% when pulling ac45ecc on SpaceFox:fix-5684_membres_ls into b41da5d on zestedesavoir:dev.

zds/forum/tests/tests.py Outdated Show resolved Hide resolved
zds/forum/tests/tests.py Outdated Show resolved Hide resolved
@SpaceFox
Copy link
Contributor Author

Attention @Situphen tu as fais un nœud dans l’arbre des commits.

Copy link
Member

@Situphen Situphen 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

@Situphen
Copy link
Member

Attention @Situphen tu as fais un nœud dans l’arbre des commits.

Ne t'inquiètes pas, Github gère bien ça et en fusionnant avec le bouton « Rebase and merge », le commit « Merge branch 'dev' into fix-5684_membres_ls » n’apparaîtra pas dans la branche dev :)

@Situphen Situphen merged commit a91d572 into zestedesavoir:dev Jan 31, 2022
@SpaceFox SpaceFox deleted the fix-5684_membres_ls branch January 31, 2022 11:04
@SpaceFox
Copy link
Contributor Author

OK, c’est curieux comme workflow.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Adapter l'affichage du forum pour les membres en lecture seule
4 participants