-
Notifications
You must be signed in to change notification settings - Fork 961
Elide manual table of contents #5338
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
Elide manual table of contents #5338
Conversation
@Rageking8 : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change. |
PRMerger Results
|
Can you review the proposed changes? Important: When the changes are ready for publication, adding a #label:"aq-pr-triaged" |
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 is good cleanup. The reason for this situation is that 'In this article' is a feature the platform team added that automatically generates links to the titles. Problem is, for the many topics we have that were basically providing this functionality manually, we now have the duplication that you noticed. Thank you for fixing these up.
#sign-off |
@TylerMSFT These changes all turn into blockers because they remove the required introductory info between the H1 and the first H2. Could you add at least a 1 sentence introduction back into these articles? #hold-off References To access the guides, you must log in with your Learn profile. If you don't have a Learn profile, follow the instructions at Manage your Microsoft Learn profile settings | Microsoft Learn to create one.
|
Learn Build status updates of commit 9e5ffa6: ✅ Validation status: passed
This comment lists only the first 25 files in the pull request. For any questions, please:
|
PRMerger Results
|
Learn Build status updates of commit 05b3e39: ✅ Validation status: passed
This comment lists only the first 25 files in the pull request. For any questions, please:
|
PRMerger Results
|
Learn Build status updates of commit a409eee: ✅ Validation status: passed
This comment lists only the first 25 files in the pull request. For any questions, please:
|
PRMerger Results
|
Learn Build status updates of commit a0a5ab9: ✅ Validation status: passed
This comment lists only the first 25 files in the pull request. For any questions, please:
|
PRMerger Results
|
Learn Build status updates of commit 66643fc: ✅ Validation status: passed
This comment lists only the first 25 files in the pull request. For any questions, please:
|
PRMerger Results
|
Learn Build status updates of commit 3407ae9: ✅ Validation status: passed
This comment lists only the first 25 files in the pull request. For any questions, please:
|
PRMerger Results
|
#sign-off I added the required text between the H1 and H2. |
Learn Build status updates of commit eee5982: ✅ Validation status: passed
This comment lists only the first 25 files in the pull request. For any questions, please:
|
PRMerger Results
|
#sign-off |
@TylerMSFT I really appreciate all the extra work put in to fix the blockers. Now merging. |
Elide all manual table of contents as they are superfluous, as seen in the following: