Skip to content
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

Merge of Hapi 7.0.0 branch into main #25

Merged
merged 72 commits into from
Mar 15, 2024
Merged

Merge of Hapi 7.0.0 branch into main #25

merged 72 commits into from
Mar 15, 2024

Conversation

KevinMayfield
Copy link
Contributor

This PR merges in changes deployed in

AWS Skunkworks Validation Tool
AWS Swagger Validator
Docker Validator

It activates this code from GitHub validation.

The main change is moving from HAPI FHIR 6 to 7.
With implied changes for swagger version and spring version (moved from 2 to 3)

Removed non FHIR Server OAS Controller
Removed non FHIR Server OAS Controller
Moved to openjdk23, java 21 and kotlin 1.9
Moved to openjdk23, java 21 and kotlin 1.9
Moved to openjdk23, java 21 and kotlin 1.9
Support for a diagnostics demo.
App deployed to AWS
Support for a diagnostics demo.
App deployed to AWS
Added configurable package parameter
fhir.igs
Added configurable package parameter
fhir.igs
Doesn't like IPS, so using EU Labs for examples
Added configurable package parameter
fhir.igs
Added configurable package parameter
fhir.igs
Added configurable package parameter
fhir.igs
# Conflicts:
#	Dockerfile
#	pom.xml
#	src/main/resources/manifest.json
Merge in ryans jdk 21 code
@ryma2fhir
Copy link
Contributor

Fails on example tests. Do we want to fix these first?

@ryma2fhir ryma2fhir self-requested a review March 15, 2024 12:05
@ryma2fhir
Copy link
Contributor

examples fail due to move from nhsd package to nhse package. To be fixed at a later date

pom.xml Outdated
Comment on lines 44 to 47
<dependency>
<groupId>com.fasterxml.jackson.module</groupId>
<artifactId>jackson-module-kotlin</artifactId>
</dependency>
Copy link
Contributor

Choose a reason for hiding this comment

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

This is a duplicate of lines 56-60 so should be removed

@ryma2fhir ryma2fhir merged commit a3d84c2 into main Mar 15, 2024
1 of 3 checks passed
@ryma2fhir ryma2fhir deleted the HAPI-7.0.0 branch March 15, 2024 12:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants