Skip to content

Fix x-state in API explorer#2894

Merged
lcawl merged 1 commit intomainfrom
x-state
Mar 13, 2026
Merged

Fix x-state in API explorer#2894
lcawl merged 1 commit intomainfrom
x-state

Conversation

@lcawl
Copy link
Contributor

@lcawl lcawl commented Mar 13, 2026

While playing in docs-builder I noticed that the code for handling x-state in the OpenAPI files was present, it just wasn't being rendered.

This PR seems to address the issue with two simple fixes.

For information about how to test this feature, refer to the docs I've crafted in #2879

Screenshots

Before

image

After

Works at both the operation-level:

image

Note that I extended that text to make sure it handled both the "since" and "lifecycle" info.
I also added one for testing purposes at the property-level:

image

Generative AI disclosure

  1. Did you use a generative AI (GenAI) tool to assist in creating this contribution?
  • Yes
  • No
  1. If you answered "Yes" to the previous question, please specify the tool(s) and model(s) used (e.g., Google Gemini, OpenAI ChatGPT-4, etc.).

Tool(s) and model(s) used: composer-1.5, claude-4.6-sonnet-medium

@lcawl lcawl added the fix label Mar 13, 2026
@lcawl lcawl marked this pull request as ready for review March 13, 2026 02:01
@lcawl lcawl requested a review from a team as a code owner March 13, 2026 02:01
@lcawl lcawl requested a review from technige March 13, 2026 02:01
Copy link
Member

@Mpdreamz Mpdreamz left a comment

Choose a reason for hiding this comment

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

Lgtm, the reason I didn't add it was because I wanted to project it to applies to and render as applies to.

We can follow up with that though

@lcawl lcawl merged commit b8d0dab into main Mar 13, 2026
31 checks passed
@lcawl lcawl deleted the x-state branch March 13, 2026 17:44
Mpdreamz pushed a commit that referenced this pull request Mar 15, 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.

2 participants