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
OAI-63: Improve contained resources and converters #86
Conversation
This pull request introduces 1 alert when merging 58fc5b5 into 914a98b - view on LGTM.com new alerts:
|
|
||
@classmethod | ||
def build_fhir_medication_frequency_extension(cls, value): | ||
# TODO: Is this ok? Value is assigned to period instead of frequency |
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.
@@ -20,13 +21,14 @@ def create(self, validated_data): | |||
return Service.objects.create(**copied_data) | |||
|
|||
def update(self, instance, validated_data): | |||
# TODO: This doesn't work, when called from contained resources update creates new entity with same uuid |
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.
Please create a ticket for this in E0 project.
@@ -25,6 +25,7 @@ def create(self, validated_data): | |||
return new_family | |||
|
|||
def update(self, instance, validated_data): | |||
# TODO: This doesn't work |
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.
Please create a ticket for this in E0 project.
Changes made to make communication with with claim AI Server: