Skip to content

[R2 SQL] Add changelog for querying R2 SQL from the dashboard#31857

Draft
Marcinthecloud wants to merge 1 commit into
productionfrom
r2-sql-studi-changelog
Draft

[R2 SQL] Add changelog for querying R2 SQL from the dashboard#31857
Marcinthecloud wants to merge 1 commit into
productionfrom
r2-sql-studi-changelog

Conversation

@Marcinthecloud

Copy link
Copy Markdown
Contributor

Summary

  • Adds a changelog entry for querying R2 Data Catalog tables with R2 SQL directly from the Cloudflare dashboard
  • Documents the built-in SQL editor: writing/running queries, schema browsing, and inspecting results and query plans
  • Includes a screenshot of the R2 SQL query editor

Screenshots (optional)

Documentation checklist

@github-actions

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

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 commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Here is my review of PR #31857.

Summary

This PR adds a changelog entry announcing the new R2 SQL dashboard query editor. It also introduces the first screenshot for the R2 SQL product (r2-sql-studio.png). The entry is well-structured, follows the changelog frontmatter schema, uses valid internal links with trailing slashes, and correctly imports components from ~/components and astro:assets.

Labels

changelog, r2-sql, documentation

Issues to Address

1. Leftover TODO in PR description

The PR body still contains:

<!-- TODO: add screenshots before requesting review -->

Since the screenshot is already included in this PR, this TODO should be removed from the PR description before merge.

2. PR checklist is unchecked

The documentation checklist (changelog entry and style guide adherence) is currently unchecked. Please check off these items if you have verified them.

3. Minor suggestions (non-blocking)

Image alt text (line 13):
alt="R2 SQL Query Editor" is acceptable, but you could make it more descriptive for accessibility:

"Cloudflare dashboard showing the R2 SQL query editor with a schema browser, SQL editor, and query results table"

This better follows the style guide recommendation to "describe what the image shows and why it matters."

What Looks Good

  • Frontmatter: Valid title, description, date, and products referencing the existing r2-sql directory entry.
  • Links: Internal docs links use root-relative paths with trailing slashes (/r2/data-catalog/, /r2-sql/). External links are fine.
  • Dashboard deep link: The https://dash.cloudflare.com/?to=/:account/data-catalog/overview pattern is used consistently across the repo.
  • Image placement: src/assets/images/r2-sql/r2-sql-studio.png follows the correct directory convention.
  • Screenshot content: Appears to use public demo data (ISS, Bluesky, weather datasets) — no obvious sensitive information.
  • Writing style: Active voice, present tense, imperative bullets, no contractions, correct UI terminology ("go to", "select").

Approval: Conditional on cleaning up the TODO comment in the PR description.

github run

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.

1 participant