Skip to content

[Radar] Add finer-grained chart granularity changelog#31200

Draft
andre-j3sus wants to merge 3 commits into
productionfrom
ajesus/radar-changelog
Draft

[Radar] Add finer-grained chart granularity changelog#31200
andre-j3sus wants to merge 3 commits into
productionfrom
ajesus/radar-changelog

Conversation

@andre-j3sus
Copy link
Copy Markdown
Contributor

Summary

Add finer-grained chart granularity changelog

Documentation checklist

@andre-j3sus andre-j3sus self-assigned this Jun 3, 2026
@andre-j3sus andre-j3sus requested review from a team as code owners June 3, 2026 09:23
@cloudflare-docs-bot
Copy link
Copy Markdown

cloudflare-docs-bot Bot commented Jun 3, 2026

Review

✅ No style-guide issues found in commit 45dd209.

Commands

Only codeowners can run commands. Post a comment with the command to trigger it.

Command Description
/review Runs a review now. Incremental if a prior review exists, full if not.
/full-review Re-reviews the entire PR diff from scratch, ignoring incremental history. Useful after a rebase, when you want a fresh review, or if the bot gets out of sync and reports issues that no longer exist.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 3, 2026

This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:

Pattern Owners
/src/assets/images/ @cloudflare/pm-changelogs, @cloudflare/product-owners
/src/content/changelog/ @cloudflare/pm-changelogs, @cloudflare/product-owners

@ask-bonk ask-bonk Bot added changelog Adds or updates changelog entries/pages. product:radar labels Jun 3, 2026
Copy link
Copy Markdown
Contributor

@ask-bonk ask-bonk Bot left a comment

Choose a reason for hiding this comment

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

1 issue needs fixing before merge.

MEDIUMsrc/content/changelog/radar/2026-06-03-radar-traffic-chart-granularity.mdx line 14: The new defaults list overlaps at exactly 3 months ("1-3 months" and "3+ months" are both inclusive of 3 months). Match the inclusive/exclusive framing used earlier in the paragraph ("1-3 month" vs. "longer than 3 months"). Posted 1 inline suggestion.

Labels applied: changelog, product:radar.

Comment thread src/content/changelog/radar/2026-06-03-radar-traffic-chart-granularity.mdx Outdated
@ask-bonk
Copy link
Copy Markdown
Contributor

ask-bonk Bot commented Jun 3, 2026

I've reviewed PR #31200 and posted my findings.

Summary: This PR adds a new Radar changelog entry about finer-grained traffic chart granularity, along with two before/after comparison screenshots. The entry is well-structured with proper frontmatter and correctly placed images.

Labels applied: changelog, product:radar

Issue flagged: I posted 1 inline suggestion on src/content/changelog/radar/2026-06-03-radar-traffic-chart-granularity.mdx line 14. The new defaults list uses overlapping boundaries:

  • **1-3 months** (inclusive of 3 months)
  • **3+ months** (also inclusive of 3 months)

This creates ambiguity for the exact 3-month view. I suggested changing **3+ months** to **Longer than 3 months** to match the inclusive/exclusive phrasing used earlier in the paragraph and eliminate the overlap.

github run

…nularity.mdx

Co-authored-by: ask-bonk[bot] <249159057+ask-bonk[bot]@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 3, 2026

@andre-j3sus andre-j3sus marked this pull request as draft June 3, 2026 15:53
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.

2 participants