Closed bashir2 closed 1 month ago
Thanks @bashir2!
This looks fine to me, but I feel like we should wait until the the fix makes it into the IG publisher before merging.
Otherwise I don't know how we can continue to build our IG successfully...
Thanks @bashir2!
This looks fine to me, but I feel like we should wait until the the fix makes it into the IG publisher before merging.
Otherwise I don't know how we can continue to build our IG successfully...
Thanks @johngrimes. Yeah that's correct; I won't merge this until my fix to the publisher tool is merged and released.
@johngrimes I think the publisher tool has been finally fixed/released and I think we can merge this PR now (the publisher tool does not produce the NPE on this change anymore). I know you are in the process of generating a release candidate for SQL-on-FHIR IG, so please merge this when the time is right.
Added two examples from Open Health Stack flattening ViewDefinitions (specifically from here). Note that the publisher tool fails on this PR with an NPE as discussed/fixed in this other PR.