Skip to content

fix: remove additional languages menu [BISERVER-14508]#6257

Merged
dcleao merged 1 commit into
masterfrom
BISERVER-14508
Jun 9, 2026
Merged

fix: remove additional languages menu [BISERVER-14508]#6257
dcleao merged 1 commit into
masterfrom
BISERVER-14508

Conversation

Copilot AI 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.

Pull request overview

Removes the “Additional Languages” entry from the PUC (Mantle) language menu, aligning the UI with BISERVER-14508 by eliminating the extra marketplace link and its now-unneeded separator insertion.

Changes:

  • Removed the additionalLanguagesMenuItem from the XUL language menu definition.
  • Removed the corresponding language menu separator insertion in MantleController that previously separated built-in languages from the “Additional Languages” link.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
user-console/src/main/resources/org/pentaho/mantle/public/xul/mantle.xul Removes the “Additional Languages” menu item from the language menu.
user-console/src/main/java/org/pentaho/mantle/client/ui/xul/MantleController.java Removes insertion of the language menu separator that was only needed for the removed menu item.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@hitachivantarasonarqube

Copy link
Copy Markdown

@buildguy

buildguy commented Jun 3, 2026

Copy link
Copy Markdown
Collaborator

👍 Frogbot scanned this pull request and did not find any new security issues.

Note:

Frogbot also supports Contextual Analysis, Secret Detection, IaC and SAST Vulnerabilities Scanning. This features are included as part of the JFrog Advanced Security package, which isn't enabled on your system.


@buildguy

buildguy commented Jun 3, 2026

Copy link
Copy Markdown
Collaborator

✅ Build finished in 5m 17s

Build command:

mvn clean verify -B -e -Daudit -Djs.no.sandbox -pl user-console

👌 All tests passed!

Tests run: 12, Failures: 0, Skipped: 0    Test Results


ℹ️ This is an automatic message

@dcleao dcleao merged commit 6a7ebcb into master Jun 9, 2026
3 checks passed
@dcleao dcleao deleted the BISERVER-14508 branch June 9, 2026 17:01
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.

5 participants