Skip to content

[EV-6233] policy screenshot updates - #3057

Merged
ctauchen merged 7 commits into
tigera:mainfrom
sunnysoumya:EV-6233
Oct 8, 2026
Merged

ctauchen merged 7 commits into
tigera:mainfrom
sunnysoumya:EV-6233

Conversation

@sunnysoumya

Copy link
Copy Markdown
Contributor

Product Version(s):

Issue:

Link to docs preview:

SME review:

  • An SME has approved this change.

DOCS review:

  • A member of the docs team has approved this change.

Additional information:

Merge checklist:

  • Deploy preview inspected wherever changes were made
  • Build completed successfully
  • Test have passed

@sunnysoumya
sunnysoumya requested a review from a team as a code owner October 6, 2026 13:46
Copilot AI balanced review requested due to automatic review settings October 6, 2026 13:46
@netlify

netlify Bot commented Oct 6, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview succeeded!

Built without sensitive environment variables

Name Link
🔨 Latest commit 83608d0
🔍 Latest deploy log https://app.netlify.com/projects/tigera/deploys/6ac779cc97d66f00083beea4
😎 Deploy Preview https://deploy-preview-3057--tigera.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 72 (🔴 down 6 from production)
Accessibility: 98 (no change from production)
Best Practices: 92 (no change from production)
SEO: 100 (no change from production)
PWA: -
View the detailed breakdown and full score reports
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify

netlify Bot commented Oct 6, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for calico-docs-preview-next ready!

Name Link
🔨 Latest commit 83608d0
🔍 Latest deploy log https://app.netlify.com/projects/calico-docs-preview-next/deploys/6ac779cc55080f00098adb46
😎 Deploy Preview https://deploy-preview-3057--calico-docs-preview-next.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

The changes omit available replacement screenshots, retain stale terminology, and inadvertently remove release-note content.

Review effort: Balanced
Findings: 4 Low severity

Open (4)
What changed in this PR

Updates Calico Cloud policy documentation for the refreshed UI.

Changes:

  • Updates policy screenshot references and terminology.
  • Removes obsolete screenshots.
  • Modifies a historical release note.
File Description
tour.mdx Updates the policy preview screenshot.
release-notes/​index.mdx Removes Helm release-note content.
staged-network-policies.mdx Removes staging and preview screenshots.
policy-tutorial-ui.mdx Updates policy UI terminology and screenshots.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread calico-cloud/network-policy/policy-tiers/policy-tutorial-ui.mdx
Comment thread calico-cloud/network-policy/policy-tiers/policy-tutorial-ui.mdx Outdated

1. From the **Policies Board**, select a staged policy and click **Edit policy**.
1. Make some edits and click **Preview**.

Comment thread calico-cloud/release-notes/index.mdx
Restores the Helm installation line in calico-cloud/release-notes/index.mdx.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings October 6, 2026 13:52
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🔵 Needs a closer look

Several available 3.24 screenshots were omitted, and one renamed section retains outdated terminology.

Review effort: Balanced
Findings: 3 Low severity

Open (3)
Resolved since last review (1)
Previously missed (1)

In code that hasn't changed since last review

Low severity Update follow-up screenshot to namespace-selector-3.24.png

calico-cloud/​network-policy/​policy-tiers/​policy-tutorial-ui.mdx:110

This workflow now mixes UI versions: the first namespace-selector screenshot is updated, but the follow-up image at line 114 still uses namespace-selector.png, although namespace-selector-3.24.png exists. Update the second reference so both screenshots depict the same UI version.

Copilot AI balanced review requested due to automatic review settings October 6, 2026 13:55

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🔵 Needs a closer look

The renamed Rules section still refers readers to the obsolete Type section.

Review effort: Balanced
Findings: 3 Low severity

Open (3)

Copilot AI balanced review requested due to automatic review settings October 6, 2026 14:12

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The updated references resolve to existing assets and the accompanying UI terminology is consistent.

Review effort: Balanced
Findings: 1 Low severity

Open (1)
Resolved since last review (2)

@ctauchen ctauchen left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM, thanks!

Copilot AI balanced review requested due to automatic review settings October 8, 2026 11:03

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

The Enterprise tutorial still directs readers to an obsolete page name beside the refreshed UI screenshot.

2 open findings

🧠 Review effort: Balanced


Give feedback about Copilot approvals in this survey to enter a drawing for a $150 gift card.

Action defines what should happen when a connection matches this rule.

![policy-tutorial-action](/img/calico-enterprise/policy-tutorial-action.png)
![policy-tutorial-action](/img/calico-enterprise/policy-tutorial-action-3.24.png)
Copilot AI balanced review requested due to automatic review settings October 8, 2026 11:08

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🔵 Needs a closer look

Several tutorial instructions remain inconsistent with the updated ingress workflow, and one updated screenshot lacks meaningful alternative text.

2 open findings
Previously missed (3)

In code that hasn't changed since last review

Medium severity Label the BE-to-FE rule as ingress

calico-cloud/​network-policy/​policy-tiers/​policy-tutorial-ui.mdx:103

The updated flow opens an ingress rule, and the described BE-to-FE traffic is inbound to the scoped FE endpoints, but the accompanying paragraph still calls this an egress rule. That reverses the rule direction for readers following the tutorial; label it as ingress (and restore the missing space before the selector).

Medium severity Label the BE-to-FE rule as ingress

calico-enterprise/​network-policy/​policy-tiers/​policy-tutorial-ui.mdx:103

The updated flow opens an ingress rule, and the described BE-to-FE traffic is inbound to the scoped FE endpoints, but the accompanying paragraph still calls this an egress rule. That reverses the rule direction for readers following the tutorial; label it as ingress (and restore the missing space before the selector).

Low severity Replace filename-like alt text with a descriptive policy preview

calico-cloud/​tutorials/​calico-cloud-features/​tour.mdx:134

The updated image keeps policy-preview as its alt text, which exposes a filename-like label rather than the screenshot’s purpose to screen-reader users. Replace it with a concise description of what the policy preview shows.

🧠 Review effort: Balanced


Give feedback about Copilot approvals in this survey to enter a drawing for a $150 gift card.

@ctauchen
ctauchen merged commit 33a4a95 into tigera:main Oct 8, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants