Skip to content

Conversation

@mikesealey
Copy link
Collaborator

@mikesealey mikesealey commented Jan 27, 2026

Description

During an interal hackathon, I found our API interface confusing when trying to hook up two endpoints.

For example, these two endpoints on the Discord API are the same path, but with a different method.
image

But the name gets show in the list, not the path, so it's confusing

image

This PR replaces the name with the path

image

Summary by cubic

Show endpoint path in the API endpoint dropdown instead of the name to make it clear when multiple methods share the same path.

Written for commit 3ec92d3. Summary will update on new commits.

@mikesealey mikesealey requested a review from melohagan January 27, 2026 10:09
@mikesealey mikesealey marked this pull request as ready for review January 27, 2026 10:09
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Copy link
Collaborator

@melohagan melohagan left a comment

Choose a reason for hiding this comment

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

I think having the operationId includes important context.

For example POST /invites (could be create invite, or search invite)

Whereas create_channel_invite is explicit about what the operation is doing.

What I could do is something similar to the role dropdown here: #17806

Where the label is the operationId, but the endpoint is also provided as a subtitle, and you can search on either.
Think that PR would need merged first so I can re-use that functionality.

@mikesealey mikesealey changed the title adds replaces endpoint name with path Replaces endpoint name with path Jan 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants