Obligation Affects dans PtSituationElement & Affects dans Consequence d'une situation #21
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
PR correspondant à l'issue etalab#3
PR pour l'issue etalab#4
TuThoThai
changed the title
|
Cette PR devra être revue par au moins deux personne parmi les suivantes : |
|
Salut
Est ce que tu pourrais me dire le processus par rapport à cela ?
Merci et bonne soirée
Envoyé à partir de Outlook pour Android<https://aka.ms/AAb9ysg>
…________________________________
From: Tu-Tho Thai ***@***.***>
Sent: Monday, November 25, 2024 7:07:19 PM
To: etalab/transport-profil-siri-fr ***@***.***>
Cc: PICAVET Francois-Xavier ***@***.***>; Mention ***@***.***>
Subject: Re: [etalab/transport-profil-siri-fr] Obligation Affects dans PtSituationElement & Affects dans Consequence d'une situation (PR #21)
Cette PR devra être revue par au moins deux personne parmi les suivantes :
* @Aurige<https://github.com/Aurige>
* @prhod<https://github.com/prhod>
* @albanpeignier<https://github.com/albanpeignier>
* @fxpicavet<https://github.com/fxpicavet>
* @Henault<https://github.com/Henault>
—
Reply to this email directly, view it on GitHub<#21 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/BJE34PEZS75WGQWH55QAVQL2CNRNPAVCNFSM6AAAAABSORA5ZGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOJYG4YDONRSGA>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
C2 – Usage restreint
|
|
Hello @fxpicavet Le processus pour relire la PR est
D'un point de formalisme normalisation, on a
|
|
Hello
Merci Tu-Tho. 2 questions :
Ne vaut il mieux pas designer les 2 relecteurs ?
Fixer une deadline ?
A+
Thierry
Bien cordialement
Thierry Hénault
Directeur de Projets
CS GROUP
***@***.***
…________________________________
De : Tu-Tho Thai ***@***.***>
Envoyé : lundi 25 novembre 2024 21:48
À : etalab/transport-profil-siri-fr ***@***.***>
Cc : HENAULT Thierry ***@***.***>; Mention ***@***.***>
Objet : Re: [etalab/transport-profil-siri-fr] Obligation Affects dans PtSituationElement & Affects dans Consequence d'une situation (PR #21)
Hello @fxpicavet<https://github.com/fxpicavet>
Le processus pour relire la PR est
* vérifier qu'elle comporte bien les éléments discutés en atelier et plénière
* que les changements sont faits au bon endroit dans le profil
* qu'il n'y a pas de typo
D'un point de formalisme normalisation, on a
* accord sur les changements des experts en atelier
* présentation des changements en plénière GT7
* aval en plénière GT7
* finalisation des changements post plénière
—
Reply to this email directly, view it on GitHub<#21 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AC3C7N2GU6BPSTBYUXAWAH32COEIXAVCNFSM6AAAAABSORA5ZGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOJZGAYDCNBWGM>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
C2 – Usage restreint
|
|
Hello @Henault, Sur les relecteurs, l'idéal serait d'avoir :
Deadline : avant le 3 décembre 2024 pour que ça soit fait pour le prochain atelier SIRI, profil France. Merci !! |
|
Hello
C'est OK pour moi pour cette PR. (Passage de 0:1 à 1:1 pour l'affect principal.
Bien cordialement
Thierry Hénault
Directeur de Projets
CS GROUP
***@***.***
…________________________________
De : Tu-Tho Thai ***@***.***>
Envoyé : lundi 25 novembre 2024 21:48
À : etalab/transport-profil-siri-fr ***@***.***>
Cc : HENAULT Thierry ***@***.***>; Mention ***@***.***>
Objet : Re: [etalab/transport-profil-siri-fr] Obligation Affects dans PtSituationElement & Affects dans Consequence d'une situation (PR #21)
Hello @fxpicavet<https://github.com/fxpicavet>
Le processus pour relire la PR est
* vérifier qu'elle comporte bien les éléments discutés en atelier et plénière
* que les changements sont faits au bon endroit dans le profil
* qu'il n'y a pas de typo
D'un point de formalisme normalisation, on a
* accord sur les changements des experts en atelier
* présentation des changements en plénière GT7
* aval en plénière GT7
* finalisation des changements post plénière
—
Reply to this email directly, view it on GitHub<#21 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AC3C7N2GU6BPSTBYUXAWAH32COEIXAVCNFSM6AAAAABSORA5ZGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOJZGAYDCNBWGM>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
C2 – Usage restreint
|
|
La 2eme partie de la PR n'est pas traitée (ie ajout (0:1) Affect dans consequence : On est d'accord ? |
albanpeignier
requested changes
Nov 26, 2024
Merci pour l'oeil de lynx, je viens de corriger ! |
Si, ce sont les commits 2 et 3 de la PR : |
albanpeignier
approved these changes
Nov 26, 2024
|
✔️ A voté |
|
Idem ok pour moi |
|
@thbar , nous avons bien les 2 validations nécessaires de @fxpicavet et @albanpeignier : tu peux lancer le merge ! 🙏 |
|
Merci à tous, c'est mergé ! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR correspondant à l'issue #3 et #4