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.

Sign up for GitHub

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

Jump to bottom

Zib-PainScore #233

Merged
merged 19 commits into from
Jul 12, 2023
Merged

Zib-PainScore #233

merged 19 commits into from
Jul 12, 2023

Conversation

Ashwin-nictiz
Copy link
Contributor

No description provided.

@github-actions
Copy link

github-actions bot commented Feb 2, 2022

Validation report for changed materials

Validation

Zib compliance

  • ✔️ Zib compliance check succeeded with 0 errors and 0 warnings.

niekvangalen
niekvangalen suggested changes Feb 8, 2022
View reviewed changes
resources/zib/zib-PainScore.xml Outdated Show resolved Hide resolved
@github-actions
Copy link

Validation report for changed materials

Validation

  • ✔️ "Validate zib profiles" succeeded with 0 fatal messages, 0 errors, 0 warnings and 0 information messages.
  • ➖ Test "Validate zib extensions" was skipped
  • ✔️ "Validate nl-core profiles" succeeded with 0 fatal messages, 0 errors, 0 warnings and 0 information messages.
  • ➖ Test "Validate nl-core extensions" was skipped
  • ➖ Test "Validate other profiles" was skipped
  • ➖ Test "Validate ConceptMaps" was skipped
  • ⚠️ "Validate other terminology" succeeded with 0 fatal messages, 0 errors, 1 warnings and 0 information messages.
  • ➖ Test "Validate examples" was skipped

Zib compliance

  • ✔️ Zib compliance check succeeded with 0 errors and 0 warnings.

@LuudSlagter LuudSlagter self-requested a review January 24, 2023 07:27
@pieter-edelman-nictiz pieter-edelman-nictiz marked this pull request as ready for review July 11, 2023 19:46
@pieter-edelman-nictiz
Copy link
Member

@LuudSlagter , zou jij nog een laatste blik op mijn commit willen werpen? Ik heb de FHIRPath constraint geherformuleerd omdat ik dit wat eleganter vind, maar wellicht zit ik ernaast.
Daarnaast ook zib-PainScore-2 asls condition toegevoegd op Observation.method. Alhoewel .method strikt genomen niet genoemd wordt in de constraint zelf, slaat de constraint wel op de minimumwaarde ten gevolge van de gebruikte methode, dus dit leek me toch logisch.

@pieter-edelman-nictiz pieter-edelman-nictiz merged commit e04fdbf into main Jul 12, 2023
1 of 2 checks passed
@pieter-edelman-nictiz pieter-edelman-nictiz deleted the zib-PainScore branch July 12, 2023 06:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@niekvangalen niekvangalen niekvangalen approved these changes

@LuudSlagter LuudSlagter LuudSlagter approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@Ashwin-nictiz @pieter-edelman-nictiz @niekvangalen @LuudSlagter @jd-nictiz