Skip to content

Conversation

@Maddy-Cloudflare
Copy link
Contributor

No description provided.

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Jan 9, 2025

Deploying cloudflare-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 62420a3
Status: ✅  Deploy successful!
Preview URL: https://35e4a12a.cloudflare-docs-7ou.pages.dev
Branch Preview URL: https://email-security-pcx-15016.cloudflare-docs-7ou.pages.dev

View logs

@Maddy-Cloudflare Maddy-Cloudflare marked this pull request as draft January 9, 2025 15:21
@cloudflare cloudflare deleted a comment from afaftrad1 Jan 23, 2025
@github-actions github-actions bot added size/m and removed size/s labels Feb 13, 2025
@github-actions github-actions bot removed the size/m label Feb 14, 2025
Copy link
Contributor

@hyperlint-ai hyperlint-ai bot left a comment

Choose a reason for hiding this comment

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

Hyperlint Review is complete!

Hyperlint Vale Style Guide Review

Reviewed 12 files and found 2 issues.

See the inline comments and suggestions below.


We are upgrading our reviewer model! Going forward, our model will learn your particular style - which comments you all accept and reject, so that it can further personalize its suggestions.

What is Hyperlint?

Hyperlint is an AI agent that helps you write, edit, and maintain your documentation. It can review, edit, and suggest fixes for your documentation.

Learn more about the Hyperlint AI reviewer and the checks that we can run on your documentation.

@Maddy-Cloudflare Maddy-Cloudflare marked this pull request as draft February 26, 2025 15:06
@hyperlint-ai
Copy link
Contributor

hyperlint-ai bot commented Mar 27, 2025

Howdy and thanks for contributing to our repo. The Cloudflare team reviews new, external PRs within two (2) weeks. If it's been two weeks or longer without any movement, please tag the PR Assignees in a comment.

We review internal PRs within 1 week. If it's something urgent or has been sitting without a comment, start a thread in the Developer Docs space internally.


PR Change Summary

Enhanced Email Security documentation with new inline deployment guides and prerequisites for various email providers.

  • Added detailed guides for setting up Email Security with inline deployment for multiple regions.
  • Introduced prerequisites for configuring Email Security with Cisco, Google Workspace, and Office 365 as MX records.
  • Included a new section on Egress IPs for Email Security deployment.

Added Files

  • src/content/docs/cloudflare-one/email-security/setup/pre-delivery-deployment/egress-ips.mdx
  • src/content/docs/cloudflare-one/email-security/setup/pre-delivery-deployment/index.mdx
  • src/content/docs/cloudflare-one/email-security/setup/pre-delivery-deployment/inline-deployment-setup.mdx
  • src/content/docs/cloudflare-one/email-security/setup/pre-delivery-deployment/inline-deployment.mdx
  • src/content/docs/cloudflare-one/email-security/setup/pre-delivery-deployment/prerequisites/cisco-email-security-mx.mdx
  • src/content/docs/cloudflare-one/email-security/setup/pre-delivery-deployment/prerequisites/cisco-mx.mdx
  • src/content/docs/cloudflare-one/email-security/setup/pre-delivery-deployment/prerequisites/gsuite-email-security-mx.mdx
  • src/content/docs/cloudflare-one/email-security/setup/pre-delivery-deployment/prerequisites/index.mdx
  • src/content/docs/cloudflare-one/email-security/setup/pre-delivery-deployment/prerequisites/office365-email-security-mx.mdx
  • src/content/partials/cloudflare-one/email-security/deployment/mx-deployment-prerequisites.mdx
  • src/content/partials/cloudflare-one/email-security/deployment/mx-deployment-values.mdx
  • src/content/partials/cloudflare-one/email-security/deployment/mx-geographic-locations.mdx

How can I customize these reviews?

Check out the Hyperlint AI Reviewer docs for more information on how to customize the review.

If you just want to ignore it on this PR, you can add the hyperlint-ignore label to the PR. Future changes won't trigger a Hyperlint review.

Note specifically for link checks, we only check the first 30 links in a file and we cache the results for several hours (for instance, if you just added a page, you might experience this). Our recommendation is to add hyperlint-ignore to the PR to ignore the link check for this PR.

Maddy-Cloudflare and others added 2 commits April 10, 2025 11:22
…ery-deployment/prerequisites/office365-email-security-mx.mdx

Co-authored-by: Jun Lee <[email protected]>
@Maddy-Cloudflare Maddy-Cloudflare marked this pull request as ready for review April 10, 2025 10:29
@Maddy-Cloudflare Maddy-Cloudflare merged commit 986f9c8 into production Apr 10, 2025
11 checks passed
@Maddy-Cloudflare Maddy-Cloudflare deleted the email-security/pcx-15016 branch April 10, 2025 10:40
RebeccaTamachiro pushed a commit that referenced this pull request Apr 21, 2025
* [Email Security] Inline deployment docs

* Fixing link

* Apply suggestions from code review

Co-authored-by: Jun Lee <[email protected]>

* Updating steps

* Adding prereq steps and egress ips

* Typo

* Fixing link

* Adding Cisco prereqs, partials, and renaming URLs

* Fixing steps

* Add submission addresses

* Remove space

* Adding graphs

* Adding Cisco diagrams

* Remove submission addresses

* Apply suggestions from code review

Co-authored-by: Jun Lee <[email protected]>

* Correcting UI component

* Update src/content/docs/cloudflare-one/email-security/setup/pre-delivery-deployment/inline-deployment-setup.mdx

Co-authored-by: Jun Lee <[email protected]>

* Correcting table

* Making steps optional

* Open Egress IPs to new page

* Update copy

* Add instructions about quarantining on MS365

* update links

* Fixing broken link + updating note

* Update src/content/docs/cloudflare-one/email-security/setup/pre-delivery-deployment/prerequisites/office365-email-security-mx.mdx

Co-authored-by: Jun Lee <[email protected]>

* Removing partial

---------

Co-authored-by: Jun Lee <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants