-
Notifications
You must be signed in to change notification settings - Fork 24
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
Move CRMI operations to Visitors #408
Move CRMI operations to Visitors #408
Conversation
….com/TahaAttari/clinical-reasoning into knowledge-artifact-visitors-adapters
<groupId>ch.qos.logback</groupId> | ||
<artifactId>logback-classic</artifactId> | ||
</dependency> |
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.
use this to test WARN level logging
add missing Extension generic params per suggestion Co-authored-by: Brenin Rhodes <brenin.rhodes@smilecdr.com>
@SuppressWarnings("unchecked") | ||
default <T extends ICompositeType & IBaseHasExtensions> List<T> getOwnedRelatedArtifacts() { |
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.
not sure why this is causing the Warning
var resources = resourceMap.computeIfAbsent(resource.fhirType(), r -> new HashMap<>()); | ||
var theId = resource.getIdElement().toUnqualifiedVersionless(); | ||
var outcome = new MethodOutcome(theId, false); |
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.
outcome should have the ID of the newly updated resource
No description provided.