Skip to content

[CASB] Adding Zoom Support - #33761

Merged
rklarsfeld merged 2 commits into
cloudflare:productionfrom
mleslie-cf:patch-10
Oct 19, 2026
Merged

rklarsfeld merged 2 commits into
cloudflare:productionfrom
mleslie-cf:patch-10

Conversation

@mleslie-cf

Copy link
Copy Markdown
Contributor

Summary

  • Update doc to cover CASB Zoom integration

Screenshots (optional)

Documentation checklist

  • [x ] Is there a changelog entry (guidelines)? If you don't add one for something awesome and new (however small) — how will our customers find out? Changelogs are automatically posted to RSS feeds, the Discord, and X.
  • [x ] The change adheres to the documentation style guide.
  • If a larger change - such as adding a new page- an issue has been opened in relation to any incorrect or out of date information that this PR fixes.
  • Files which have changed name or location have been allocated redirects.

@cloudflare-docs-bot

cloudflare-docs-bot Bot commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor

AI Review

✅ Reviewed 8880016 · 2 findings · incremental

👉 Fix in your agent 👈
Fix the following review findings in PR #33761 (https://github.com/cloudflare/cloudflare-docs/pull/33761).

Before making changes, review each finding and present a brief summary table:
- For each finding, state whether you agree, disagree, or need clarification.
- If you disagree, explain why.
- If you need clarification before deciding, ask those questions.
- Then share your plan for which issues to tackle and in what order.

After triaging, follow this order:
1. Post a comment on this PR for any findings you are skipping, with the finding ID and your reasoning.
2. Then commit the fixes for the legitimate findings.

The comment must come before the commit because the next review reads PR comments.

## Code Review
- C-16a32d7c | src/content/docs/cloudflare-one/integrations/cloud-and-saas/zoom.mdz | New page uses `.mdz` extension instead of `.mdx`
  The file is added as `zoom.mdz` (confirmed at the PR head; `zoom.mdx` does not exist). The CI `Check for invalid file extensions` step in `.github/workflows/ci.yml` fails on any file under `src/content` that is not `.mdx`, `.md`, `.json`, `.yml`, `.yaml`, or `.txt`, so this PR fails CI, and the docs content collection does not ingest `.mdz` files, so the page will not render. Every other CASB integration page in this directory uses `.mdx`, and the `casb/security-findings` partial renders its RSS link as `.../cloud-and-saas/${slug}.mdx.atom`, which points to a path that will not exist while the file is named `.mdz`.

## Conventions
- V-047291ab | PR | Description omits that this PR adds a new page
  The only changed file, `src/content/docs/cloudflare-one/integrations/cloud-and-saas/zoom.mdx`, is a new page (status `added`, +226 lines). The description says only "Update doc to cover CASB Zoom integration" and does not mention that a new page is being added.

Code Review

Skipped: no code changes to review.

  • src/content/docs/cloudflare-one/integrations/cloud-and-saas/zoom.mdz · New page uses .mdz extension instead of .mdx

    The file is added as zoom.mdz (confirmed at the PR head; zoom.mdx does not exist). The CI Check for invalid file extensions step in .github/workflows/ci.yml fails on any file under src/content that is not .mdx, .md, .json, .yml, .yaml, or .txt, so this PR fails CI, and the docs content collection does not ingest .mdz files, so the page will not render. Every other CASB integration page in this directory uses .mdx, and the casb/security-findings partial renders its RSS link as .../cloud-and-saas/${slug}.mdx.atom, which points to a path that will not exist while the file is named .mdz.

Conventions

  • PR · Description omits that this PR adds a new page

    The only changed file, src/content/docs/cloudflare-one/integrations/cloud-and-saas/zoom.mdx, is a new page (status added, +226 lines). The description says only "Update doc to cover CASB Zoom integration" and does not mention that a new page is being added.

Commands
  • /review Run a review now.
  • /full-review Review the entire PR diff.
  • /disable-auto-review Stop automatic reviews.
  • /rebase Rebase against production.

@grstnhbr
grstnhbr enabled auto-merge (squash) October 5, 2026 22:04
@mvvmm mvvmm mentioned this pull request Oct 12, 2026
2 tasks
auto-merge was automatically disabled October 19, 2026 19:51

Head branch was pushed to by a user without write access

@mleslie-cf mleslie-cf left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Correcting file extension issue

@rklarsfeld
rklarsfeld enabled auto-merge October 19, 2026 22:06
@rklarsfeld
rklarsfeld added this pull request to the merge queue Oct 19, 2026
Merged via the queue into cloudflare:production with commit 6f2a385 Oct 19, 2026
9 checks passed
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.

3 participants