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

Preferences for contribution printing #46

Merged
merged 2 commits into from Mar 8, 2020
Merged

Preferences for contribution printing #46

merged 2 commits into from Mar 8, 2020

Conversation

nbehier
Copy link
Contributor

@nbehier nbehier commented Mar 7, 2020

Ajout d'une préférence Galette pour que les membres puissent imprimer leurs contributions

Preferences for contribution printing
5e3813b 
Ajout préférence pour que les membres puissent imprimer leurs contributions
@nbehier
Copy link
Contributor Author

nbehier commented Mar 7, 2020

@trasher Peux-tu me dire si cette contribution te semble aller dans le bon sens ? Il manque l'internationnalisation, mais sinon je crois que tout y est.

trasher
trasher reviewed Mar 7, 2020
View reviewed changes
Copy link
Member

@trasher trasher left a comment

Choose a reason for hiding this comment

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

Pourquoi ajouter une préférence finalement ? Est-il vraiment souhaitable que les adhérents ne puissent pas accéder à ces documents ? (toute limitation actuelle est purement fortuite ;))

@nbehier
Copy link
Contributor Author

nbehier commented Mar 8, 2020

@trasher :-D
OK, je simplifie le code et rend le PDF accessible à tous alors !

Print for all + security
3271dd9 
- L'impression est disponible à tous
- ajout d'un message d'erreur si demande d'impression d'une contribution inexistante ou pour un autre membre
@nbehier
Copy link
Contributor Author

nbehier commented Mar 8, 2020

@trasher J'ai donc supprimé la partie pour les préférences, rendant l'impression des contributions accessible à tous. J'en ai profité pour ajouter un message d'erreur si on demande l'impression d'une contribution inaccessible.

trasher
trasher approved these changes Mar 8, 2020
View reviewed changes
Copy link
Member

@trasher trasher left a comment

Choose a reason for hiding this comment

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

Ça me semble tout à fait bon, merci :)

@trasher trasher merged commit 9719882 into galette:develop Mar 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@trasher trasher trasher approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
2 participants
@nbehier @trasher