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

Un utilisateur en attente de validation d'une asso peut commencer à rédiger un article #342

Open
ppom0 opened this issue Mar 7, 2020 · 3 comments
Labels
good first issue Good for newcomers not-urgent-issue Has to be fixed, but doesn't prevent from using the app. Not to be mistaken for not-urgent-feature UX Issue in User eXperience

Comments

@ppom0
Copy link
Contributor

ppom0 commented Mar 7, 2020

Le bouton "rédiger un article" est visible pour les utilisateurs non validés. Il faudrait simplement le cacher.
Ce sera peut être réglé par la correction de cette issue de @noeamiot.

@MercierCorentin MercierCorentin added good first issue Good for newcomers not-urgent-issue Has to be fixed, but doesn't prevent from using the app. Not to be mistaken for not-urgent-feature UX Issue in User eXperience labels Mar 8, 2020
@noeamiot
Copy link
Member

La correction de l'issue #322 ne corrige pas cette issue car actuellement aucune condition n'est vérifiée pour afficher ce bouton.
Je peux m'en charger mais je me demandais, sur le principe qui, dans une association a le droit de rédiger un article ?

@ppom0
Copy link
Contributor Author

ppom0 commented Mar 10, 2020

Héhé... Il me semble que n'importe quel membre peut poster quelque chose. De toute manière, c'est bien implémenté côté back, donc l'article ne pourra pas être créé si l'utilisateur n'a pas les droits. À tester avec les données du seed.

@MercierCorentin
Copy link
Contributor

MercierCorentin commented Mar 10, 2020

Salut, seul les membres dont le rôle à la permission article peuvent publier pour une asso ;)
Pour les articles users c'est différent mais c'est pas encore en prod.
C'est déterminé, pour chaque modèle implémentant l'interface CanHaveArticles, par la méthode isArticleManageableBy.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers not-urgent-issue Has to be fixed, but doesn't prevent from using the app. Not to be mistaken for not-urgent-feature UX Issue in User eXperience
Projects
None yet
Development

No branches or pull requests

3 participants