Skip to content

UNOMI-953: Backport packaging bundle (Phase 2 PR 2)#785

Merged
sergehuber merged 2 commits into
masterfrom
UNOMI-953-packaging-bundle
Jun 29, 2026
Merged

UNOMI-953: Backport packaging bundle (Phase 2 PR 2)#785
sergehuber merged 2 commits into
masterfrom
UNOMI-953-packaging-bundle

Conversation

@sergehuber

Copy link
Copy Markdown
Contributor

Summary

  • Reorder distribution/src/main/feature/feature.xml so unomi-cxs-privacy-extension-services and plugin features install before REST, privacy, conditions, lists, and geonames (matches unomi-3-dev / kar/ dependency graph).
  • Preserve master’s #774 unomi-plugins-advanced-conditions wiring in all four distribution features.
  • Port extension micro-deltas from unomi-3-dev: null-safe condition validation in UserListServiceImpl, and Javadoc </p> fixes in ScriptMetadata / GroovyActionsService (UNOMI-897).
  • Intentionally skipped: GroovyActionDispatcher (master UNOMI-873: Add request tracing and web-servlets extraction #775 tracing wins), GroovyActionsServiceImplTest (master UNOMI-881: Add service and extension unit tests (part 2/2) #776), geonameEntry.json (newline-only diff).

Test plan

  • mvn -pl extensions/lists-extension/services compile
  • mvn -pl distribution -am package -DskipTests
  • Karaf boot smoke: verify distribution features resolve and privacy/REST bundles start in correct order
  • List metadata query with and without a condition filter

Reorder distribution Karaf features to install privacy-extension-services before plugins and REST, while preserving master’s advanced-conditions wiring. Port list query condition validation and Groovy action Javadoc fixes from unomi-3-dev without reverting master tracing improvements.
Close the paragraph before the list block; HTML auto-closes <p> when <ul> starts, which caused doclint "unexpected end tag" failures.
@sergehuber sergehuber merged commit ba8a4c2 into master Jun 29, 2026
6 checks passed
@sergehuber sergehuber deleted the UNOMI-953-packaging-bundle branch June 29, 2026 04:37
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.

1 participant