Skip to content
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

DOC-7154 Revising the alerts taxonomy #3568

Merged
merged 19 commits into from
Aug 24, 2021

Conversation

paperclypse
Copy link
Contributor

@paperclypse paperclypse commented Aug 19, 2021

This PR is for generating the Gatsby preview for a proof of concept (which doesn't happen with a draft PR).

The taxonomy revision looks good, but I still need to add redirects and move all the files around.

@github-actions github-actions bot added this to Hero to triage in Docs PRs and Issues Aug 19, 2021
@paperclypse paperclypse moved this from Hero to triage to Drafts in Docs PRs and Issues Aug 19, 2021
@gatsby-cloud
Copy link

gatsby-cloud bot commented Aug 19, 2021

Gatsby Cloud Build Report

docs-website-develop

🎉 Your build was successful! See the Deploy preview here.

Build Details

View the build logs here.

🕐 Build time: 39m

@paperclypse
Copy link
Contributor Author

For some reason, these docs won't load in the Gatsby preview or the local build:
Create, edit, or find alert policies
Create conditions
Intro to NerdGraph API and Alerts
Use the API (section)
Baseline alert conditions

I imagine I made some pathing mistakes with these.

@paperclypse paperclypse self-assigned this Aug 24, 2021
@paperclypse paperclypse added from_tw Identifies issues/PRs from Tech Docs writers content requests related to docs site content labels Aug 24, 2021
@paperclypse paperclypse moved this from Drafts to Hero: To do in Docs PRs and Issues Aug 24, 2021
@paperclypse paperclypse moved this from Hero: To do to Writer needs peer edit (Hero or any TW can take) in Docs PRs and Issues Aug 24, 2021
@paperclypse
Copy link
Contributor Author

Ready for a peer review! This is DOC-7154

@paperclypse paperclypse changed the title First try at revising the alerts taxonomy DOC-7154 Revising the alerts taxonomy Aug 24, 2021
@paperclypse paperclypse marked this pull request as draft August 24, 2021 16:51
@paperclypse paperclypse marked this pull request as ready for review August 24, 2021 16:52
@github-actions github-actions bot requested a review from mmfred August 24, 2021 16:52
@mmfred
Copy link
Contributor

mmfred commented Aug 24, 2021

@paperclypse This is working as expected. Just one thing: this link in the Learn about Alerts area has kind of weird behavior in the NAV (the page opens fine): Technical rules and limits
It works, the Nav just collapses and the reader loses their place.

mmfred
mmfred previously approved these changes Aug 24, 2021
@paperclypse
Copy link
Contributor Author

Thanks @mmfred!

@paperclypse paperclypse moved this from Writer needs peer edit (Hero or any TW can take) to Waiting on SME/Blocked in Docs PRs and Issues Aug 24, 2021
@paperclypse paperclypse merged commit 03ad1a9 into develop Aug 24, 2021
@paperclypse paperclypse deleted the alerts-taxonomy-reorganization branch August 24, 2021 21:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content requests related to docs site content from_tw Identifies issues/PRs from Tech Docs writers
Projects
Docs PRs and Issues
Waiting on SME/Blocked
Development

Successfully merging this pull request may close these issues.

None yet

2 participants