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

Délégation de compte: ETQ gestionnaire je peux contacter les gestionnaires des groupes parents #9846

Conversation

seb-by-ouidou
Copy link
Contributor

@seb-by-ouidou seb-by-ouidou commented Dec 13, 2023

@seb-by-ouidou seb-by-ouidou force-pushed the feature-ouidou/admin_creation_delegation_gestionnaire_contact_parent_gestionnaires branch 2 times, most recently from 8aa0439 to 8bc9d46 Compare January 19, 2024 14:52
@seb-by-ouidou seb-by-ouidou force-pushed the feature-ouidou/admin_creation_delegation_gestionnaire_contact_parent_gestionnaires branch from 8bc9d46 to 1811383 Compare February 2, 2024 15:29
@seb-by-ouidou seb-by-ouidou marked this pull request as ready for review February 5, 2024 08:28
@seb-by-ouidou seb-by-ouidou force-pushed the feature-ouidou/admin_creation_delegation_gestionnaire_contact_parent_gestionnaires branch 3 times, most recently from 1811383 to 22e53e5 Compare February 5, 2024 13:05
@colinux colinux changed the title Feature ouidou/admin creation delegation gestionnaire contact parent gestionnaires Délégation de compte: ETQ gestionnaire je peux contacter les gestionnaires des groupes parents Feb 9, 2024
Copy link
Member

@colinux colinux left a comment

Choose a reason for hiding this comment

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

Quelques retours, surtout un sujet avec view trop compliquée et dure à suivre

end

def create
@commentaire = @groupe_gestionnaire.commentaire_groupe_gestionnaires.create(commentaire_params.merge(sender_id: @last_commentaire.sender_id, sender_type: @last_commentaire.sender_type, gestionnaire: current_gestionnaire))
@commentaire = @last_commentaire.groupe_gestionnaire.commentaire_groupe_gestionnaires.create(commentaire_params.merge(sender_id: @last_commentaire.sender_id, sender_type: @last_commentaire.sender_type, gestionnaire: current_gestionnaire))
Copy link
Member

Choose a reason for hiding this comment

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

peut-être que ça passe pour être plus concis :

Suggested change
@commentaire = @last_commentaire.groupe_gestionnaire.commentaire_groupe_gestionnaires.create(commentaire_params.merge(sender_id: @last_commentaire.sender_id, sender_type: @last_commentaire.sender_type, gestionnaire: current_gestionnaire))
@commentaire = @last_commentaire.groupe_gestionnaire.commentaire_groupe_gestionnaires.create(commentaire_params.merge(sender: @last_commentaire.sender, gestionnaire: current_gestionnaire))

Copy link
Member

Choose a reason for hiding this comment

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

Beaucoup de logique compliquée dans le vue, y compris du SQL. Ce serait envisageable de remonter ça dans le controller,(ou à défaut dans le modèle ou component…) ?

Copy link
Member

Choose a reason for hiding this comment

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

OK on a vu le fix dans la PR d'après !

@seb-by-ouidou seb-by-ouidou force-pushed the feature-ouidou/admin_creation_delegation_gestionnaire_contact_parent_gestionnaires branch from 22e53e5 to f02c0da Compare February 13, 2024 09:38
@seb-by-ouidou seb-by-ouidou force-pushed the feature-ouidou/admin_creation_delegation_gestionnaire_contact_parent_gestionnaires branch from d19ca25 to 08cffda Compare February 13, 2024 10:19
@colinux colinux enabled auto-merge February 13, 2024 14:28
@colinux colinux added this pull request to the merge queue Feb 13, 2024
Merged via the queue into demarches-simplifiees:main with commit 48644ae Feb 13, 2024
16 checks passed
@colinux colinux deleted the feature-ouidou/admin_creation_delegation_gestionnaire_contact_parent_gestionnaires branch February 13, 2024 14:54
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.

2 participants