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

[ML] Update inference endpoint and connector flyout for read-only scenario #213586

Open
alvarezmelissa87 opened this issue Mar 7, 2025 · 0 comments
Labels
Team:ML Team label for ML (also use :ml)

Comments

@alvarezmelissa87
Copy link
Contributor

Currently if I am viewing the details of an inference endpoint, but I do not have permission to edit these, the fields show as disabled and a save button is still shown (disabled).

We should change these form fields to be read-only rather than disabled so they are visible to screen-readers and the user has the option to copy the text in them.

We should remove the save button as this can never be enabled by something the user can do within the flyout, so it's unnecessary to show.

This change would effect all Connector forms. So we need to make sure we are aligned with Response Ops folks and the full effect of this change.

List known restrictions:

This form uses our Rich Configurable Fields framework so it will impact the behavior at the framework level.

Are there any existing features or workflows that are directly affected by this request?

Inference Endpoints
Connectors
AI Connector

@alvarezmelissa87 alvarezmelissa87 added the Team:ML Team label for ML (also use :ml) label Mar 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team:ML Team label for ML (also use :ml)
Projects
None yet
Development

No branches or pull requests

1 participant