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

Le conseiller peut connaitre le résultat par sous-niveau clea #1786

Conversation

marouria
Copy link
Contributor

@marouria marouria commented Dec 12, 2024

Capture d’écran 2024-12-18 à 09 55 43

@marouria marouria changed the title 📝 Cadrage Le conseiller peut connaitre le résultat par sous-niveau clea Dec 12, 2024
@marouria marouria force-pushed the EVA-113-le-conseiller-peut-connaitre-le-resultat-par-sous-niveau-clea branch 2 times, most recently from ad25598 to 4fc13e0 Compare December 17, 2024 08:12
app/models/evenement.rb Outdated Show resolved Hide resolved
app/models/evenement.rb Outdated Show resolved Hide resolved
@cprodhomme cprodhomme force-pushed the EVA-113-le-conseiller-peut-connaitre-le-resultat-par-sous-niveau-clea branch from 4fc13e0 to 2199bdc Compare December 20, 2024 19:12
Ces scopes ne sont pas des scopes. Un scope ActiveRecord par définition doit retourner une collection du model dans lequel il est, ici ce n'était pas le cas, il s'agissait d'un array de questions trier en ruby
J'ai volontairement appelé ce composant BadgeProfil plutot que Badge, car en web, le composant badge peut avoir une autre signification

Exemple : https://mui.com/material-ui/react-badge/

Je ne suis pas convaincu par le vert mais je pense qu'il faut une maquette
Pour éviter les erreurs et les répétitions
@cprodhomme
Copy link
Contributor

Capture d’écran 2024-12-20 à 21 11 11

@cprodhomme cprodhomme force-pushed the EVA-113-le-conseiller-peut-connaitre-le-resultat-par-sous-niveau-clea branch from e96bb40 to 28a7fa8 Compare December 23, 2024 15:44
@cprodhomme cprodhomme marked this pull request as ready for review December 23, 2024 15:44
@Guitguitou Guitguitou merged commit 704ca44 into develop Dec 23, 2024
5 checks passed
@Guitguitou Guitguitou deleted the EVA-113-le-conseiller-peut-connaitre-le-resultat-par-sous-niveau-clea branch December 23, 2024 16:15
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