-
Notifications
You must be signed in to change notification settings - Fork 8
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
Fix assos names #348
base: develop
Are you sure you want to change the base?
Fix assos names #348
Conversation
@Cathaiste @noeamiot @cesar-richard |
{asso.short_description && <p>{asso.short_description}</p>} | ||
<span className="mt-4">{asso.type && asso.type.description}</span> | ||
<ReactMarkdown className="my-3 text-justify" source={asso.description} /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je trouve l'idée cool mais il faudra, sur le menu d'édition pour les assos, bien préciser qu'est-ce qui correspond à quoi. En gros j'ai peur que ce soit pas clair pour les assos mais si on fait bien le menu d'édition de l'asso ça devrait être OK.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, je fais une Doc utilisateur avec cette PR alors! On fera bien le menu d'édition et on pourra éventuellement mettre un lien vers la Doc dedans.
La doc est là en attente là https://gitlab.utc.fr/simde/documentation/-/merge_requests/4 |
Je voudrais bien review la doc mais j'ai pas les droits pour voir la mr (ni le repo enft) |
Noé à approuvé, faut juste que je fasse les changements ! Un deuxième regard est intéressant ! |
@MercierCorentin je repasse la dessus, tu pourras faire une update au moins du statut de la PR stp ? :) |
Je m'y repenche d'ici fin janvier max ! |
Fixes #112