feat - update BYOK disable tip by xinyi-gong · Pull Request #140 · microsoft/copilot-for-eclipse · GitHub
Skip to content

feat - update BYOK disable tip - #140

Merged
Sheng Chen (jdneo) merged 3 commits into
mainfrom
tori/changeByokDescription
May 7, 2026
Merged

Sheng Chen (jdneo) merged 3 commits into
mainfrom
tori/changeByokDescription

Conversation

@xinyi-gong

Copy link
Copy Markdown
Member

BYOK now supports Copilot Business and Enterprise users when enabled by their organization (see https://github.com/microsoft/copilot-language-server-internal/pull/733). This PR updates the tip to reflect this change.

@xinyi-gong
xinyi-gong marked this pull request as ready for review May 7, 2026 04:27
Comment thread CHANGELOG.md Outdated
Copilot AI review requested due to automatic review settings May 7, 2026 04:50

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

Note

Copilot was unable to run its full agentic suite in this review.

Updates the BYOK (custom models) disabled-tip text and corresponding message key to align the Eclipse UI messaging with BYOK availability for Business/Enterprise org-managed settings, and records the change in the changelog.

Changes:

  • Replaced the BYOK “preview disabled” tip/message key with a new “disabled by enterprise settings” tip.
  • Updated Java NLS bindings and UI usage to reference the new message key.
  • Added a changelog entry announcing BYOK support for Business/Enterprise users.

Reviewed changes

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

File Description
com.microsoft.copilot.eclipse.ui/src/com/microsoft/copilot/eclipse/ui/preferences/messages.properties Replaces the BYOK disabled-tip string/key.
com.microsoft.copilot.eclipse.ui/src/com/microsoft/copilot/eclipse/ui/preferences/Messages.java Renames the NLS constant to match the updated properties key.
com.microsoft.copilot.eclipse.ui/src/com/microsoft/copilot/eclipse/ui/preferences/ByokPreferencePage.java Switches the UI tip reference to the renamed message constant.
CHANGELOG.md Adds an entry documenting BYOK support for Business/Enterprise users.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@jdneo
Sheng Chen (jdneo) merged commit 68b5a6f into main May 7, 2026
4 checks passed
@jdneo
Sheng Chen (jdneo) deleted the tori/changeByokDescription branch May 7, 2026 05:11
@jdneo

Copy link
Copy Markdown
Member

Andrey Loskutov (@iloveeclipse) BYOK for enterprise and business is unblocked now.

@iloveeclipse

Copy link
Copy Markdown
Contributor

Great. Will there be a nightly build to test soon?

@jdneo

Copy link
Copy Markdown
Member

@jdneo Sheng Chen (jdneo) mentioned this pull request May 7, 2026
24 tasks done
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