-
Notifications
You must be signed in to change notification settings - Fork 24.8k
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
docs: api pages improvements #55388
docs: api pages improvements #55388
Conversation
* add link for source files * add deprecation message for methods * add usage notes for methods Fixes angular#55196 Fixes angular#52641
Path should have a leading slash.
Deployed adev-preview for 3d2aff8 to: https://ng-dev-previews-fw--pr-angular-angular-55388-adev-prev-el9mru6m.web.app Note: As new commits are pushed to this pull request, this link is updated after the preview is rebuilt. |
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.
LGTM
Thanks for adding the deprecation notice back in! |
Going to wait for @atscott's review before setting it to merge state! |
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.
🥇
This PR was merged into the repository by commit 4d61501. |
Path should have a leading slash. PR Close #55388
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
docs: improvements for API pages
Fixes #55196
Fixes #52641
Demo :