Skip to content

Conversation

@MartinWheelerMT
Copy link
Collaborator

@MartinWheelerMT MartinWheelerMT commented May 19, 2025

  • Update gradle resources to version 8.0.0
  • Breaking model changes now means that the Parameter.getParameter(name) now returns a Type which cannot now be cast to a StringType. We can work around this by finding the ParametersParameterComponent manually and then casting it to retrieve the value.
  • Removed unused method extractValueOrThrow
  • Due to Parameter changes in the latest version, there is now a slightly different error phrasing in the error returned when a parameter is missing or empty. Integration tests have been updated to reflect this.

* Update gradle resources to version `8.0.0`
* Breaking model changes now means that the `Parameter.getParameter(name)` now returns a `Type` which cannot now be cast to a `StringType`. We can work around this by finding the `ParametersParameterComponent` manually and then casting it to retrieve the value.
MartinWheelerMT and others added 2 commits May 20, 2025 10:13
* Due to Parameter changes in the latest version, there is now a slightly different error phrasing in the error returned when a parameter is missing or empty.  Integration tests have been updated to reflect this.
@github-actions
Copy link

github-actions bot commented May 20, 2025

Images built and published to ECR using a Build Id of PR-78-660edcb

@MartinWheelerMT MartinWheelerMT enabled auto-merge (squash) May 20, 2025 14:22
adjust modifiers to private
}

@SneakyThrows
public String extractValueOrThrow(String name, Supplier<? extends Throwable> exception) {
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This method was unused so it was removed rather than updated

@MartinWheelerMT MartinWheelerMT merged commit b999030 into develop May 21, 2025
6 checks passed
@MartinWheelerMT MartinWheelerMT deleted the update-hapi-fhir-r4-to-8.0.0 branch May 21, 2025 14:48
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