Skip to content

Comments

mise à jour de la page consultation offre et le modifier _elements n'est pas implémenté#415

Merged
sga-esante merged 11 commits intomainfrom
cp-consultation-offre
Jul 17, 2025
Merged

mise à jour de la page consultation offre et le modifier _elements n'est pas implémenté#415
sga-esante merged 11 commits intomainfrom
cp-consultation-offre

Conversation

@sga-esante
Copy link
Contributor

@sga-esante sga-esante commented Jun 23, 2025

Closes #414

Description des changements

*mise à jour de specifications_techniques_2.md : Scénario 1: ajout d'un avertissement
*mise à jour de specifications_techniques_2.md : Scénarios 1bis et 1ter: ajout d'une note importante
*mise à jour de specifications_techniques_2.md : Scénarios 2, 3 et 4: suppression du label "draft"
*_elements n'est pas implémenté: mise à jour des CS Server et Client ainsi que de la page modifiers.md

Preview

https://ansforge.github.io/IG-fhir-repertoire-offre-ressources-sante/cp-consultation-offre/ig pour prévisualiser l'IG d'une branche

Impact API ROR

Pas d'impact, il s'agit d'éléments déjà implémentés en 4.1 ou antérieur

@sga-esante sga-esante requested a review from sdemeyANS June 23, 2025 16:44
@sdemeyANS sdemeyANS requested a review from nriss July 10, 2025 08:29
@sga-esante sga-esante requested a review from nriss July 10, 2025 12:56
@sga-esante
Copy link
Contributor Author

Prise en compte des remarques de @nriss pour mise à jour de specifications_techniques_2.md et point supplémentaire traité: "_elements n'est pas implémenté: mise à jour des CS Server et Client ainsi que de la page modifiers.md"

@sga-esante sga-esante changed the title mise à jour de specifications_techniques_2.md mise à jour de la page consultation offre et le modifier _elements n'est pas implémenté Jul 10, 2025
@nriss
Copy link
Member

nriss commented Jul 15, 2025

C'est top! C'est un moindre mal de dire les défaillances de l'IG et de signaler les perspectives d'évolution.
Je mets un bémol sur le fait que ça fait des mois que l'on essaie de diriger le guide dans le droit chemin et d'augmenter la qualité sans succès, et demande à voir son évolution/refacto.

Je laisse @sdemeyANS valider si c'est ok pour lui :)

Copy link
Collaborator

@sdemeyANS sdemeyANS left a comment

Choose a reason for hiding this comment

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

Voici mes retours.

#### Scénario 1 bis : Extraction complète asynchrone <code><span style="color: #ff0000;">draft</span></code>

<p style="background-color: #ffcccc; border:1px solid grey; padding: 5px; max-width: 790px;">
<b>Note importante:</b> Ce scénario est implémenté dans la version actuelle du ROR mais nous le maintenons à l'état draft car il ne respecte pas les points suivants de la spécification du bulkdata <a>https://hl7.org/fhir/uv/bulkdata/export.html</a> : <br>
Copy link
Collaborator

Choose a reason for hiding this comment

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

du coup ce n'est pas exactement ce scenario qui est implémenté dans le ROR ? Peut être dire que ce besoin est couvert met ne respecte pas le standard...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Il me semble que c'est bien ce que j'indique déjà et j'énumère les points pour lesquels le standard n'est pas respecté.


<p style="background-color: #ffcccc; border:1px solid grey; padding: 5px; max-width: 790px;">
<b>Note importante:</b> Ce scénario est implémenté dans la version actuelle du ROR mais nous le maintenons à l'état draft car il ne respecte pas les points suivants de la spécification du bulkdata <a>https://hl7.org/fhir/uv/bulkdata/export.html</a> : <br>
- le paramètre <i>_outputFormat</i> ne supporte pas <i>application/fhir+ndjson</i>, les valeurs possibles sont <i>application/fhir+json</i> ou <i>application/json</i><br>
Copy link
Collaborator

Choose a reason for hiding this comment

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

à la place de l'italique mettre des balises <code> lignes 134 à 136

Copy link
Contributor Author

Choose a reason for hiding this comment

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

c'est fait

Copy link
Contributor Author

@sga-esante sga-esante left a comment

Choose a reason for hiding this comment

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

J'ai pris en compte les demandes.
Si certains commentaires ou autres corretifs ne vous vont pas, n'hésitez pas à faire directement le changement. L'objectif étant de fusionner les PR le 28/7.

#### Scénario 1 bis : Extraction complète asynchrone <code><span style="color: #ff0000;">draft</span></code>

<p style="background-color: #ffcccc; border:1px solid grey; padding: 5px; max-width: 790px;">
<b>Note importante:</b> Ce scénario est implémenté dans la version actuelle du ROR mais nous le maintenons à l'état draft car il ne respecte pas les points suivants de la spécification du bulkdata <a>https://hl7.org/fhir/uv/bulkdata/export.html</a> : <br>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Il me semble que c'est bien ce que j'indique déjà et j'énumère les points pour lesquels le standard n'est pas respecté.


<p style="background-color: #ffcccc; border:1px solid grey; padding: 5px; max-width: 790px;">
<b>Note importante:</b> Ce scénario est implémenté dans la version actuelle du ROR mais nous le maintenons à l'état draft car il ne respecte pas les points suivants de la spécification du bulkdata <a>https://hl7.org/fhir/uv/bulkdata/export.html</a> : <br>
- le paramètre <i>_outputFormat</i> ne supporte pas <i>application/fhir+ndjson</i>, les valeurs possibles sont <i>application/fhir+json</i> ou <i>application/json</i><br>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

c'est fait

@sga-esante sga-esante requested review from nriss and sdemeyANS July 17, 2025 09:15
Copy link
Contributor Author

@sga-esante sga-esante left a comment

Choose a reason for hiding this comment

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

Application des modifications vu ensemble le 17/7

@sdemeyANS sdemeyANS added this to the next release milestone Jul 17, 2025
Copy link
Member

@nriss nriss left a comment

Choose a reason for hiding this comment

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

ok :)

@sga-esante sga-esante merged commit 32df1db into main Jul 17, 2025
1 check passed
github-actions bot pushed a commit that referenced this pull request Jul 17, 2025
…est pas implémenté (#415)

* mise à jour de specifications_techniques_2.md

* amélioration présentation des notes

* Modification 1er phrase avertissement pour $export

* _elements n'est pas implementé actuellement

* remplacement balise i par balide code

* ajout commentaire à ror-consommateur.fsh

* Ajout d'un saut de ligne

* Update ror-serveur.fsh

* Update input/pagecontent/specifications_techniques_2.md

* Update input/pagecontent/specifications_techniques_2.md

* Update input/pagecontent/specifications_techniques_2.md 32df1db
@sga-esante sga-esante deleted the cp-consultation-offre branch July 29, 2025 14:46
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.

[CP]: Mise à jour de la page Consultation des données capacitaires (specifications_techniques_1)

3 participants