feat: add/edit tool filtering #338
Conversation
- disable all - http method type
|
|
@mdrazak2001 is attempting to deploy a commit to the Speakeasy Team on Vercel. A member of the Team first needs to authorize it. |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
mdrazak2001
changed the title
mdrazak2001
changed the title
| ]; | ||
|
|
||
| const enabledFilter = ( | ||
| <MultiSelect |
There was a problem hiding this comment.
May need to run prettier on this file
| {allEnabled ? ( | ||
| <X className="w-4 h-4" /> | ||
| ) : ( | ||
| <div className="flex gap-2 justify-end"> |
There was a problem hiding this comment.
Nice feature!
I think a more common pattern would be a checkbox in the header row inline with the other checkboxes?
There was a problem hiding this comment.
@farazcsk : okie, i think thatll be more elegant, but 1 quick qn,
are we planning on a single checkbox? if yes say the checkbox is checked, and some tools are deselected, would the checkbox get unchecked? and to deselect all tools would the user need to do 2 clicks, unchecked(if we assume it gets unchecked on partial tool deslection) -> check(manual) -> uncheck(manual)
2 participants
Adding Filtering methods on add/edit tool screen:
Fixes: #308