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

Block model config UI to eliminate ambiguity #1

Closed

Conversation

LaChimere
Copy link

@LaChimere LaChimere commented May 31, 2023

This PR is to block the model config UI, which contains a list of models to choose, to eliminate the ambiguous experience for users when using Azure OpenAI Services.

Changed file:

  • app/components/model-config.tsx

You can merge the owner's latest main branch to remove other file changes.

@realDuang
Copy link
Owner

try to reopen it to trigger file diff

@realDuang realDuang closed this Jun 1, 2023
@realDuang realDuang reopened this Jun 1, 2023
@realDuang
Copy link
Owner

realDuang commented Jun 1, 2023

@LaChimere I have no idea why are there still have so many changes after diff updated... Could you please try to create a new branch based by this branch and cherry-pick your changes? Thanks!

Have checked for feature changes and agree that masking is the best option before azure openai opening the model selection api.

@LaChimere
Copy link
Author

@LaChimere I have no idea why are there still have so many changes after diff updated... Could you please try to create a new branch based by this branch and cherry-pick your changes? Thanks!

Have checked for feature changes and agree that masking is the best option before azure openai opening the model selection api.

I will check it later today. Thanks.

@LaChimere LaChimere closed this Jun 1, 2023
@LaChimere LaChimere deleted the lachimere/azure_openai branch June 1, 2023 13:49
realDuang pushed a commit that referenced this pull request Jun 30, 2023
realDuang pushed a commit that referenced this pull request Sep 15, 2023
realDuang pushed a commit that referenced this pull request Nov 9, 2023
realDuang pushed a commit that referenced this pull request Jan 2, 2024
…ader

Update api.ts to set proper header "Accept"
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.

2 participants