Skip to content

Fix assos names#348

Open
MercierCorentin wants to merge 4 commits intodevelopfrom
fix/assos-names
Open

Fix assos names#348
MercierCorentin wants to merge 4 commits intodevelopfrom
fix/assos-names

Conversation

@MercierCorentin
Copy link
Copy Markdown
Contributor

Fixes #112

@MercierCorentin
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Contributor Author

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

@r0one
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Member

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

@MercierCorentin
Copy link
Copy Markdown
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