-
-
Notifications
You must be signed in to change notification settings - Fork 15
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
Implemented STU3 #20
Implemented STU3 #20
Conversation
Thanks, I will need some time to look up the STU3 spec and such and check everything, as it is just my free time :D |
Take all the time you need, we will be using our fork until this is merged anyway |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alright, found time quite early actually, so was way faster.
I still need to check a few things, but it already looks quite good! I mean much of it is just a copy (still need to find a way to get rid of the code duplication, any nice ideas?).
Only missing checks:
- Completeness of test examples
- Correctness of JSON schema
- Generated code matches the current generate crate
- xtask test actually runs through
So everything should be fine ^^
A few remarks I have:
Huh, apparently you aren't using the most up to date definitions of STU3. Do you mind if I switch to the newest downloads version? Do I have access to push here? :D I would of course only add a commit so your contribution would remain the same. There is also (also done on my local branch):
|
Ouch the new version does not include the |
The version of STU3 I used was 3.0.2, released on Feb 21, 2017. That should be the latest one right? Like you, I had to edit a bunch of things in the spec JSON files to get it working. Some things I changed:
Let me add that!
Haha I had the same OCD kicking in. Problem is that
Whoops, I don't use VSCode so I missed that. Added it now. |
Hmm the
No I am using rustfmt too ^^ So I will need to live with it :D Alright, I think that's it, I need to take a final look and then I will merge it, thank you very much! |
On the version history page (https://hl7.org/fhir/history.html), in that big table, I think the correct latest version is called "3.0.0" and released on Mar 21, 2017. While at the top of the page they refer to Release 3 STU as 3.0.2, released on Feb 21, 2017. Not sure how a newer version 3.0.2 has an earlier release date than the older version 3.0.0, but here we are haha. Anyway, I used the link to 3.0.2 in the "major milestones" to get to the final STU3 spec and got the specification from the download page of that version: https://hl7.org/fhir/stu3/downloads.html. I can't imagine that that is an older version. The version.info file came from there as well. I have no idea why that file is then suddenly referring to a version 3.0.1, but it's all pretty confusing. Feel free to change it if you want. |
Alright, then it should be the newest :D Weird. No, totally fine then, thanks! |
Great library! We needed STU3 support so I used your generator to generate the STU3 model and added an STU3 client with the appropriate tests. All tests should be passing, let me know what you think.