Repository navigation
[CASB] Adding Zoom Support - #33761
Conversation
AI Review✅ Reviewed 👉 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 ReviewSkipped: no code changes to review.
Conventions
Commands
|
Head branch was pushed to by a user without write access
mleslie-cf
left a comment
There was a problem hiding this comment.
Correcting file extension issue
Summary
Screenshots (optional)
Documentation checklist