Skip to content

Fix/recursive primitive extensions - #62

Open
dmitryashutov wants to merge 2 commits into
mainfrom
fix/recursive-primitive-extensions
Open

Fix/recursive primitive extensions#62
dmitryashutov wants to merge 2 commits into
mainfrom
fix/recursive-primitive-extensions

Conversation

@dmitryashutov

@dmitryashutov dmitryashutov commented Jul 24, 2026 •
edited
Loading

Copy link
Copy Markdown
Member
  • Convert primitive extensions (_display, _text, _valueString, etc.) recursively anywhere in the questionnaire tree for both FCE→FHIR and FHIR→FCE
  • Avoids copying nested structures like answerOption[].valueCoding._display and initial[]._valueString as-is
  • Extends translation fixtures to cover those nested cases

@dmitryashutov
Process primitive extensions recursively across the questionnaire tree.
914acfc 
Nested fields like answerOption valueCoding._display and initial._valueString were copied as-is; walk the structure so FCE and FHIR shapes convert in both directions.
@dmitryashutov
Extend translation fixtures with nested primitive extension examples.
a56a36c 
Cover initial._valueString and answerOption valueCoding._display translations in both FCE and FHIR questionnaire fixtures.
@dmitryashutov dmitryashutov self-assigned this Jul 24, 2026
@dmitryashutov
dmitryashutov requested a review from ruscoder July 27, 2026 06:34
ruscoder
ruscoder approved these changes Jul 28, 2026
View reviewed changes

@ruscoder ruscoder left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ruscoder ruscoder ruscoder approved these changes

Assignees

@dmitryashutov dmitryashutov

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@dmitryashutov @ruscoder