Skip to content

docs: document one-click email unsubscribe and dbt review dev-mode link#11353

Open
keydunov wants to merge 1 commit into
masterfrom
claude/gallant-ramanujan-rs2v22
Open

docs: document one-click email unsubscribe and dbt review dev-mode link#11353
keydunov wants to merge 1 commit into
masterfrom
claude/gallant-ramanujan-rs2v22

Conversation

@keydunov

Copy link
Copy Markdown
Member

Check List

  • Tests have been run in packages where changes have been made if available
  • Linter has been run for changed code
  • Tests for the changes have been added if not covered yet
  • Docs have been added / updated if required

Description of Changes Made

Found while sweeping recent cubejs-enterprise changes for undocumented
customer-facing behavior:

  • Scheduled refreshes (docs/explore-analyze/scheduled-refreshes.mdx):
    dashboard-refresh notification emails now include a one-click
    Unsubscribe link (RFC 8058, no sign-in required) — added a note next to
    the existing in-app subscribe/unsubscribe toggle, since this also covers
    recipients with no Cube account (e.g. embed users).
  • dbt integration (docs/integrations/dbt.mdx): the "ready to review"
    email link now drops edit-capable recipients directly into development mode
    on the synced branch instead of a read-only diff — noted alongside the
    existing review-notifications description.

Both are small, surgical additions to existing pages, per
docs-mintlify/CLAUDE.md.


Generated by Claude Code

- Scheduled refreshes: note that notification emails include a one-click
  unsubscribe link that works without signing in to Cube, including for
  recipients without a Cube account.
- dbt integration: note that the sync review email link now drops
  edit-capable recipients straight into development mode, while
  view-only recipients still see a read-only diff.
@claude

claude Bot commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

Claude finished @keydunov's task in 49s —— View job


Code Review

Verdict: LGTM ✅ — 0 issues. Docs-only, well-scoped.

Review details
  • Read PR diff
  • Verify internal link target exists and uses repo convention
  • Check tone/placement against docs-mintlify/CLAUDE.md

Files

  • docs-mintlify/docs/explore-analyze/scheduled-refreshes.mdx (+5) — adds a paragraph about the one-click unsubscribe link, correctly placed after the existing subscribe/unsubscribe editor-toggle discussion. Calls out the embed/no-Cube-account case explicitly, which is the reason this note is worth adding.
  • docs-mintlify/docs/integrations/dbt.mdx (+4/-1) — extends the existing "ready to review" email paragraph with the edit-vs-view-only landing behavior. Links to /docs/data-modeling/dev-mode.

Checks performed

  • Link target /docs/data-modeling/dev-mode resolves to docs-mintlify/docs/data-modeling/dev-mode.mdx and matches the convention used across the repo (reference/configuration/environment-variables.mdx:1121, admin/monitoring/audit-log.mdx:84, etc.).
  • Both edits are surgical additions to existing pages — no new pages, no navigation changes required — consistent with docs-mintlify/CLAUDE.md guidance.
  • No Mintlify component misuse; prose only.
  • RFC 8058 is mentioned in the PR body but intentionally omitted from the user-facing doc, which is the right call for this audience.

Nothing to flag on quality, security, performance, or testing (N/A for docs).

• Branch: `claude/gallant-ramanujan-rs2v22`

@mintlify

mintlify Bot commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
cubed3 🟢 Ready View Preview Jul 25, 2026, 12:14 AM

💡 Tip: Enable Workflows to automatically generate PRs for you.

@mintlify

mintlify Bot commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
cubed3 🟡 Building Jul 25, 2026, 12:14 AM

💡 Tip: Enable Workflows to automatically generate PRs for you.

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.

2 participants