-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Prefix operation ids with parent id #1245
Prefix operation ids with parent id #1245
Conversation
@BBlowers thank you for your PR! Unfortunately, this is a breaking change. Old URLs will stop working. Would you have some time add support for old Let me know, I can pick this up. |
@RomanHotsiy good point. Supporting both |
@RomanHotsiy I've added the I've kept the changes to a minimum as I'm not too familiar with typescript. |
@RomanHotsiy is this PR likely to get another look soon? |
Hey @BBlowers, Sorry for the radio silence. I will review and merge it this week. This looks good. I just have to test if it doesn't break our other tools. |
@RomanHotsiy How did the tests go? Is this okay to be merged? |
Oh, sorry, not really. I haven't tested it yet. Thanks for pinging. I am now adding it to the team sprint board so we prioritize it. If it is not merged mid-next week - feel free to ping me again! Sorry, busy months for me :( |
I just tested it out and there is some conflict with our internal tool. It should be fixed early next week and I will merge after. |
any news? |
@RomanHotsiy Any update on this pr? |
cc0c845
to
3c4ece9
Compare
any news ? |
Hi folks! Huge delay here but we'll merge it soon! We've finally got some time here. It will be included in 2.0.0 release in about a month. |
@BBlowers Hi! Thank you for your contribution to redoc. |
Currently, operations that have multiple tags appear multiple times on the page as desired, however, their containing elements share the same id. This means when using the side menu to navigate the user is always taken to the earliest occurrence.
To demonstrate I've added the
pet
tag to the "Returns pet inventories by status" operation in this gif. Clicking on the "Returns pet inventories by status" operation in the store section takes the user to the operation under the pet sectionA solution to this is to prefix operation ids with their parent's id.