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

Feature ouidou/admin creation delegation gestionnaire answer administrateur #9645

Conversation

seb-by-ouidou
Copy link
Contributor

@seb-by-ouidou seb-by-ouidou commented Oct 26, 2023

@seb-by-ouidou seb-by-ouidou marked this pull request as draft November 6, 2023 14:09
@seb-by-ouidou seb-by-ouidou force-pushed the feature-ouidou/admin_creation_delegation_gestionnaire_answer_administrateur branch from 1ee8413 to 8968000 Compare December 27, 2023 16:41
@seb-by-ouidou seb-by-ouidou force-pushed the feature-ouidou/admin_creation_delegation_gestionnaire_answer_administrateur branch from 8968000 to ec89f47 Compare January 11, 2024 10:25
@seb-by-ouidou seb-by-ouidou marked this pull request as ready for review January 11, 2024 10:30
@seb-by-ouidou seb-by-ouidou force-pushed the feature-ouidou/admin_creation_delegation_gestionnaire_answer_administrateur branch from ec89f47 to 562a52a Compare January 17, 2024 12:07
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.

Hello, on a détaillé dans la PR, dans le fond on préfère favoriser la réutilisation et adaptation de code qu'on a déjà plutôt que dupliquer (et surtout maintenir) du nouveau code assez proche dans l'esprit. On parle plus spécifiquement du MessageComponent.

Dispo si nécessaire

cc @krichtof

@seb-by-ouidou
Copy link
Contributor Author

toutes les modifications demandées ont eté effectuées. Comme cela concernait aussi les prochaines PR en attente qui arriverons ensuite , j'ai créé une nouvelle PR qui part de la derniere des PR qui sont en attente
#9926

Screenshot 2024-01-22 at 11-42-56 demarches-simplifiees syn
Screenshot 2024-01-22 at 11-42-30 demarches-simplifiees syn
Screenshot 2024-01-22 at 11-43-21 demarches-simplifiees syn

@seb-by-ouidou seb-by-ouidou force-pushed the feature-ouidou/admin_creation_delegation_gestionnaire_answer_administrateur branch 2 times, most recently from abca82d to cdc3529 Compare January 22, 2024 12:09
@seb-by-ouidou seb-by-ouidou force-pushed the feature-ouidou/admin_creation_delegation_gestionnaire_answer_administrateur branch from cdc3529 to 0bb189d Compare January 22, 2024 13:36
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.

OK, avec les refacto dans #9926 , merci !

@colinux colinux enabled auto-merge January 22, 2024 14:49
@colinux colinux added this pull request to the merge queue Jan 22, 2024
Merged via the queue into demarches-simplifiees:main with commit 716e4bb Jan 22, 2024
16 checks passed
@colinux colinux deleted the feature-ouidou/admin_creation_delegation_gestionnaire_answer_administrateur branch January 22, 2024 15:55
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