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

Fix assos names #348

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from
Open

Fix assos names #348

wants to merge 4 commits into from

Conversation

MercierCorentin
Copy link
Contributor

Fixes #112

@MercierCorentin
Copy link
Contributor Author

MercierCorentin commented Mar 19, 2020

@Cathaiste @noeamiot @cesar-richard
J'ai besoin d'un avis là dessus! Aussi bien sur la technique que sur l'usage. Si c'est accepté, je vois avec notre resp com pour faire passer le message aux assos!

noeamiot
noeamiot previously approved these changes Mar 19, 2020
Comment on lines 204 to 206
{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} />
Copy link
Member

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.

Copy link
Contributor Author

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.

@MercierCorentin
Copy link
Contributor Author

La doc est là en attente là https://gitlab.utc.fr/simde/documentation/-/merge_requests/4

@r0one
Copy link
Contributor

r0one commented Apr 16, 2020

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)

@MercierCorentin
Copy link
Contributor Author

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 !

@cesar-richard
Copy link
Member

@MercierCorentin je repasse la dessus, tu pourras faire une update au moins du statut de la PR stp ? :)

@MercierCorentin
Copy link
Contributor Author

Je m'y repenche d'ici fin janvier max !

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