-
Notifications
You must be signed in to change notification settings - Fork 22.5k
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
Move Web/API/Houdini to Web/API/Houdini_APIs | fix the reference bug #31677
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the update @skyclouds2001. The change largely makes sense to me, although I've made some suggestions that would make the link text more consistent.
I also think we should get @estelle's blessing on this one — it was her that moved the Houdini landing page under /APIs
recently, as part of the project to remove the /Guide
tree.
Co-authored-by: Chris Mills <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's just get this one merged; I don't think there is anything too controversial in here, thinking about it some more. Sorry for the delay.
Thanks! |
Description
in https://developer.mozilla.org/en-US/docs/Web/API, the Houdini is a broken link:
this is because the overview key does not match the summary page
sso choose to move the api summary page, also match the behavior of other apis (especially Sensor APIs)
Motivation
Additional details
Related issues and pull requests