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

add: CpS statements for account PTDATA663 #334

Merged
merged 8 commits into from
Nov 30, 2023

Conversation

f-peverali
Copy link
Contributor

Contributor Pull Request

adds statements to CapabilityStatement regarding Account

Description

Addend statements according to Account I- Interaktion (https://simplifier.net/guide/Implementierungsleitfaden-ISiK-Basismodul-Stufe-3/ImplementationGuide-markdown-Datenobjekte-Datenobjekte-Abrechnungsfall?version=current)

Motivation and Context

PTDATA663

How has this been tested?

Not tested yet

Snippets or Screenshots (if necessary):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • My code follows the code style of this IG / specification.
  • My change requires a change to the documentation or narrative (intend) of the IG.
  • I have already updated the documentation / narrative (intend) accordingly.

## Weitere Festlegungen auf Profilebene im CapabilityStatement

Festlegungen zu Anforderungen auf Profilebene werden im [ISIKCapabilityStatement](https://simplifier.net/guide/Implementierungsleitfaden-ISiK-Basismodul-Stufe-3/ImplementationGuide-markdown-CapabilityStatement?version=current) geführt.
Copy link
Contributor

Choose a reason for hiding this comment

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

funktionieren eigentlich auch relative Markdown-Links?

Copy link
Contributor Author

@f-peverali f-peverali Nov 24, 2023

Choose a reason for hiding this comment

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

gute frage! @alexzautke , @ylboerner geht das? wenn nicht, wäre das ein neues Feature in Simplifier wert. Die Absoluten URLs verursachen viel unnötigen Wartungsaufwand...

@simoneOnFhir
Copy link
Contributor

Falls irgendwer mal Zeit für einen Monkey-Job hat, würde ich vorschlagen (eventuell lässt sich das mit Hilfe von GoFSH sogar automatisieren?), die Lesbarkeit des Codes mittels indentation zu verbessern, also z.B.:

* rest.resource[=].searchParam[+]
  * extension
    * url = $capabilitystatement-expectation
    * valueCode = #SHALL
  * name = "status"
  * definition = "http://hl7.org/fhir/SearchParameter/Account-status"
  * type = #token
* rest.resource[=].searchParam[+]
  * extension
    * url = $capabilitystatement-expectation

Ich würde vorschlagen, denn Branch ohne weiteres Review zu mergen, sofern SUSHI fehlerfrei durchläuft und den Review dann in Simplifier zu machen.

@f-peverali
Copy link
Contributor Author

f-peverali commented Nov 29, 2023

Falls irgendwer mal Zeit für einen Monkey-Job hat, würde ich vorschlagen (eventuell lässt sich das mit Hilfe von GoFSH sogar automatisieren?), die Lesbarkeit des Codes mittels indentation zu verbessern, also z.B.:

* rest.resource[=].searchParam[+]
  * extension
    * url = $capabilitystatement-expectation
    * valueCode = #SHALL
  * name = "status"
  * definition = "http://hl7.org/fhir/SearchParameter/Account-status"
  * type = #token
* rest.resource[=].searchParam[+]
  * extension
    * url = $capabilitystatement-expectation

Ich würde vorschlagen, denn Branch ohne weiteres Review zu mergen, sofern SUSHI fehlerfrei durchläuft und den Review dann in Simplifier zu machen.

Monkey job bitte nicht. Wenn jemand weiß wie es automatisiert geht, dann gerne.

Bitte auch nicht einfach in TC_3.0.1 merge (oder gar main) ich habe eine Sandbox für sowas... verlinke ich im Ticket

…reifendeFestlegungen_BestaetigungsrelevanteSysteme.md
…reifendeFestlegungen_BestaetigungsrelevanteSysteme.md
…reifendeFestlegungen_BestaetigungsrelevanteSysteme.md
@ylboerner ylboerner merged commit ac4d2c7 into TC_3.0.1 Nov 30, 2023
1 check passed
@ylboerner ylboerner deleted the fix/account-in_CpS-PTDATA663 branch November 30, 2023 11:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants