Skip to content

Update UI gradle configuration and integration - #4649

Merged
malliaridis merged 4 commits into
apache:mainfrom
malliaridis:feature/admin-ui-configuration
Jul 21, 2026
Merged

Update UI gradle configuration and integration#4649
malliaridis merged 4 commits into
apache:mainfrom
malliaridis:feature/admin-ui-configuration

Conversation

@malliaridis

Copy link
Copy Markdown
Contributor

Description

With the latest updates some of the functions used by the gradle configuration in the UI module are deprecated. This PR addresses these deprecations and updates the way it is included in the webapp (similar to js-client).

Solution

This PR replaces deprecations in the build.gradle.kts file of the UI module. It also relocates the JVM files from the custom-named desktop JVM sourceset to the default jvm sourceset.

Additionally, it updates the logic used for integrating the generated artifacts into the webapp during builds by following the same approach the js-client is included, with exposed configurations.

Tests

Only manual tests have been executed and confirmed that the project builds and the UI artifacts are included correctly.

Checklist

Please review the following and check all that apply:

  • I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
  • I have created a Jira issue and added the issue ID to my pull request title.
  • I have given Solr maintainers access to contribute to my PR branch. (optional but recommended, not available for branches on forks living under an organisation)
  • I have developed this patch against the main branch.
  • I have run ./gradlew check.
  • I have added tests for my changes.
  • I have added documentation for the Reference Guide
  • I have added a changelog entry for my change

@github-actions github-actions Bot added documentation Improvements or additions to documentation dependencies Dependency upgrades tool:build admin-ui labels Jul 21, 2026
@malliaridis
malliaridis force-pushed the feature/admin-ui-configuration branch from 824fa6f to bc10ca7 Compare July 21, 2026 11:48

@dsmiley dsmiley left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I reviewed the build aspects. Thank you for addressing the TODO I put there to do this like js-client.

@malliaridis

Copy link
Copy Markdown
Contributor Author

Thank you for addressing the TODO I put there to do this like js-client.

Thank you for providing both the todo and an example of how to do it. :D

@malliaridis
malliaridis merged commit 4101b81 into apache:main Jul 21, 2026
5 checks passed
malliaridis added a commit that referenced this pull request Jul 21, 2026
@malliaridis malliaridis added this to the 10.x milestone Jul 21, 2026
@malliaridis
malliaridis deleted the feature/admin-ui-configuration branch July 21, 2026 21:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

admin-ui dependencies Dependency upgrades documentation Improvements or additions to documentation no-changelog tool:build

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants