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

Update msg for WCA model not found error and create a new error no org linked #1309

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

TamiTakamiya
Copy link
Collaborator

Update the error message displayed for the error that WCA model was not found. Old msg was:

Could not find a Model Id for IBM watsonx Code Assistant. Please contact your administrator.

and the new message is

Your organization does not have an IBM watsonx Code Assistant model configured. Contact your Red Hat organization administrator to configure a model, or provide a model id override in your VSCode Ansible extension settings

With this change, a new error is added for the case that the user is not linked to an organization because the error__wca_model_id_not_found error was issued for that case as well. The error is separated with the new message:

Your user Id is not linked to an organization Id. Please contact your administrator.

This requires the change on the server side. The server side PR will be created separately.

Copy link
Contributor

@manstis manstis left a comment

Choose a reason for hiding this comment

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

LGTM 👍

src/features/lightspeed/errors.ts Outdated Show resolved Hide resolved
@TamiTakamiya
Copy link
Collaborator Author

@robinbobbitt @goneri Would you take a look at this again? I need to have approvals from the reviewers who have requested changes. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

None yet

4 participants