Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Hide LLM dropdown for enterprise users #2393

Merged
merged 2 commits into from
Dec 15, 2023

Conversation

philipp-spiess
Copy link
Contributor

c.f. https://sourcegraph.slack.com/archives/C05AGQYD528/p1702648955261399?thread_ts=1702602272.959489&cid=C05AGQYD528

The current experience is poor since it does not show proper names and appears just broken. Also you can not change the model for enterprise setups anyways (we have a hard check for this on the backend).

Test plan

  • sg start locally
Screenshot 2023-12-15 at 14 57 51

@philipp-spiess philipp-spiess requested review from chwarwick and a team December 15, 2023 14:10
@philipp-spiess philipp-spiess merged commit b8910dc into main Dec 15, 2023
13 checks passed
@philipp-spiess philipp-spiess deleted the ps/hide-llm-dropdown-on-enterprise branch December 15, 2023 14:55
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.

None yet

3 participants