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.

Sign up for GitHub

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

Jump to bottom

feat(Alerts & AI): Added new intro doc DOC-7658 #5807

Merged
merged 10 commits into from
Jan 28, 2022
Merged

feat(Alerts & AI): Added new intro doc DOC-7658 #5807

merged 10 commits into from
Jan 28, 2022

Conversation

paperclypse
Copy link
Contributor

This replaces the old-style landing page intro we've been using.

@github-actions
Copy link

Hi @paperclypse 👋
Thanks for your pull request! Your PR is in a queue, and a writer will take a look soon. We generally publish small edits within one business day, and larger edits within three days.

Gatsby Cloud will automatically generate a preview of your request, and will comment with a link when the preview is ready (usually 20 to 30 minutes).

@github-actions github-actions bot added this to Hero to triage in Docs PRs and Issues Jan 26, 2022
@paperclypse paperclypse self-assigned this Jan 26, 2022
@paperclypse paperclypse added content requests related to docs site content from_tw Identifies issues/PRs from Tech Docs writers labels Jan 26, 2022
@rhetoric101 rhetoric101 self-requested a review January 26, 2022 00:44
@paperclypse paperclypse changed the title feat(Alerts & AI): Added new intro docDOC-7658 feat(Alerts & AI): Added new intro doc DOC-7658 Jan 26, 2022
@gatsby-cloud
Copy link

gatsby-cloud bot commented Jan 26, 2022 •
edited

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: 25m

rhetoric101
rhetoric101 previously approved these changes Jan 26, 2022
View reviewed changes
Copy link
Contributor

@rhetoric101 rhetoric101 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@paperclypse, here are some suggestions. I like how this tries to guide users through the UI!

src/content/docs/alerts-applied-intelligence/overview.mdx Outdated Show resolved Hide resolved
src/content/docs/alerts-applied-intelligence/overview.mdx Show resolved Hide resolved
src/content/docs/alerts-applied-intelligence/overview.mdx Show resolved Hide resolved
src/content/docs/alerts-applied-intelligence/overview.mdx Outdated Show resolved Hide resolved
src/content/docs/alerts-applied-intelligence/overview.mdx Outdated Show resolved Hide resolved
src/content/docs/alerts-applied-intelligence/overview.mdx Outdated Show resolved Hide resolved
src/content/docs/alerts-applied-intelligence/overview.mdx Outdated Show resolved Hide resolved
src/content/docs/alerts-applied-intelligence/overview.mdx Outdated Show resolved Hide resolved
src/content/docs/alerts-applied-intelligence/overview.mdx Outdated Show resolved Hide resolved
src/content/docs/alerts-applied-intelligence/overview.mdx Show resolved Hide resolved
@urbiz-nr urbiz-nr moved this from Hero to triage to Waiting on TW to merge in Docs PRs and Issues Jan 26, 2022
@paperclypse paperclypse marked this pull request as ready for review January 26, 2022 19:37
rhetoric101
rhetoric101 previously approved these changes Jan 26, 2022
View reviewed changes
Copy link
Contributor

@rhetoric101 rhetoric101 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@paperclypse, these changes look great! I just found one small spacing issue.

src/content/docs/alerts-applied-intelligence/overview.mdx Outdated Show resolved Hide resolved
@paperclypse
Copy link
Contributor Author

I'm going to run this change by a couple people before merging it in.

rhetoric101
rhetoric101 previously approved these changes Jan 28, 2022
View reviewed changes
Copy link
Contributor

@rhetoric101 rhetoric101 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@paperclypse, this looks good! I just inserted a few minor comments!

src/content/docs/alerts-applied-intelligence/overview.mdx Outdated Show resolved Hide resolved
src/content/docs/alerts-applied-intelligence/overview.mdx Outdated Show resolved Hide resolved
src/content/docs/alerts-applied-intelligence/overview.mdx Outdated Show resolved Hide resolved
src/content/docs/alerts-applied-intelligence/overview.mdx Show resolved Hide resolved
@paperclypse paperclypse merged commit 7132445 into develop Jan 28, 2022
@paperclypse paperclypse deleted the doc-7658-new-alerts-ai-intro branch January 28, 2022 21:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@HenryTech HenryTech HenryTech left review comments

@rhetoric101 rhetoric101 rhetoric101 left review comments

@bradleycamacho bradleycamacho Awaiting requested review from bradleycamacho

Assignees

@paperclypse paperclypse

Labels
content requests related to docs site content from_tw Identifies issues/PRs from Tech Docs writers
Projects
Docs PRs and Issues
Waiting on TW to merge
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@paperclypse @rhetoric101 @HenryTech