hapifhir
Feat/extra resource loading from npm
This feature is backwards compatible and introduces the option to load e.g. example packages from IG's based on configuration
This functionality would probably make sense to just roll into the base PackageInstallationSpec. I'd rather not have this project implementing novel features not actually found in the base project, that's hard for support.
Sure ... I can make a PR to the base getting the additionalResourceFolders added the PackageInstallationSpec.
@jamesagnew whats your take on the refactored version of the PostConstruct I've done here in https://github.com/hapifhir/hapi-fhir-jpaserver-starter/pull/784/files#diff-0f42a12718125be49d417ad9c402f5f2cecddfb7cb0ecfde364ad09132fa2a99R21 as mentioned in https://github.com/hapifhir/hapi-fhir-jpaserver-starter/pull/758
and btw - the same thing I did on https://github.com/hapifhir/hapi-fhir-jpaserver-starter/pull/784/files#diff-cca286d85a1c14aea8b529430ecd47aa4a803b53f4c7816deb7d602bd2af3268R11 can be done for a loooooong range of interceptor logic currently handled in the already way-too-big https://github.com/hapifhir/hapi-fhir-jpaserver-starter/blob/792f520f629004504f30f88a96d1f5a6e584105b/src/main/java/ca/uhn/fhir/jpa/starter/common/StarterJpaConfig.java#L262 method (I'm not pointing fingers here as I most likely authored a lot of it myself).
@jamesagnew I'll move the additional sources logic once its been merged in here - acceptable for you? Then we can merge this into https://github.com/hapifhir/hapi-fhir-jpaserver-starter/pull/785 and remove it from the starter as soon as it can be resolved from a snapshot in core.
@jamesagnew / @dotasek - I've isolated the feature set now into https://github.com/hapifhir/hapi-fhir/pull/6802. Let me know if thats the feature set you had in mind, before I start writing tests and all.
Which would be after https://github.com/hapifhir/hapi-fhir-jpaserver-starter/pull/842
This Pull Request has failed the formatting check
Please run mvn spotless:apply or mvn clean install -DskipTests to fix the formatting issues.
You can automate this auto-formatting process to execute on the git pre-push hook, by installing pre-commit and then calling pre-commit install --hook-type pre-push. This will cause formatting to run automatically whenever you push.
This Pull Request has failed the formatting check
Please run mvn spotless:apply or mvn clean install -DskipTests to fix the formatting issues.
You can automate this auto-formatting process to execute on the git pre-push hook, by installing pre-commit and then calling pre-commit install --hook-type pre-push. This will cause formatting to run automatically whenever you push.
This Pull Request has failed the formatting check
Please run mvn spotless:apply or mvn clean install -DskipTests to fix the formatting issues.
You can automate this auto-formatting process to execute on the git pre-push hook, by installing pre-commit and then calling pre-commit install --hook-type pre-push. This will cause formatting to run automatically whenever you push.