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

EVA-113 le conseiller peut connaitre le resultat par sous niveau clea 3 #1795

Conversation

cprodhomme
Copy link
Contributor

No description provided.

@cprodhomme cprodhomme force-pushed the EVA-113-le-conseiller-peut-connaitre-le-resultat-par-sous-niveau-clea-3 branch from 1afdd40 to c8fa788 Compare January 2, 2025 15:50
Copy link
Member

@etienneCharignon etienneCharignon left a comment

Choose a reason for hiding this comment

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

Salut Clément, J'ai fait une relecture, mais malheureusement, je n'aurai pas le temps de tester aujourd'hui.

app/assets/stylesheets/admin/composants/_badge_profil.scss Outdated Show resolved Hide resolved
config/locales/views/badge_profil.yml Outdated Show resolved Hide resolved
app/assets/stylesheets/admin/composants/_badge_profil.scss Outdated Show resolved Hide resolved
app/assets/stylesheets/admin/composants/_badge_profil.scss Outdated Show resolved Hide resolved
J'ai volontairement mis warning pour le orange car c'est ce qu'il représente le mieux.
On pourra se poser la question si c'est la bonne couleur par défaut à mettre (spoiler: non)

J'ai mis un title pour le success mais par pour le warning car sinon cela peut être contre intuitif
@cprodhomme cprodhomme force-pushed the EVA-113-le-conseiller-peut-connaitre-le-resultat-par-sous-niveau-clea-3 branch from 5018e00 to cfa46fc Compare January 3, 2025 09:55
@Marionvel Marionvel merged commit 82c36c1 into develop Jan 3, 2025
5 checks passed
@Marionvel Marionvel deleted the EVA-113-le-conseiller-peut-connaitre-le-resultat-par-sous-niveau-clea-3 branch January 3, 2025 10:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants