docs: Excel/Sheets add-in refresh-all, drag-to-Filters, and folder deletion - #11765
Conversation
…letion - The pivot builder's Rows/Columns/Measures drag now also accepts Filters, with an unapplied filter shown greyed until given a value. - An unsaved exploration can be deleted from the editor's own menu. - Document workbook homes as not limited to one placed exploration per sheet. - "Refresh all" runs as a tracked, resumable operation with a progress footer and per-row states, and a banner now warns before Refresh overwrites cells edited by hand. - Relabel workbook-home placement states to match the current UI: "Out of date" (was "Stale") and "Not saved" (was "Unsaved"), no longer shown as colored tags. - Deleting a non-empty folder now deletes its contents with confirmation, instead of being blocked until it's emptied. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01GuYqSYDJ9NP1xSsSuFyC7c
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Automations to automatically generate PRs for you. |
|
Claude finished @vasilev-alex's task in 1m 59s —— View job Docs review — 1 medium, 4 low. Excel/Sheets parity is clean; one factual slip in
|
| # | Severity | File | Issue |
|---|---|---|---|
| 1 | Medium | folders.mdx:128 |
Lists reports as folder content that deletion removes. Contradicts the page's own "What can go in a folder" (Workbooks / Dashboards / Explorations) and sharing.mdx:104. Per the taxonomy in docs-mintlify/CLAUDE.md, a report lives inside a workbook tab, not directly in a folder. |
| 2 | Low | folders.mdx:127-135 |
Emphasis inverted — the irreversible cascade is plain prose while the <Warning> carries a permission precondition that prevents data loss. Also, "Full access to everything inside the folder" implies per-item permissions the page's own table doesn't define. |
| 3 | Low | microsoft-excel.mdx:95-98, google-sheets.mdx:87-90 |
"use the funnel buttons … the same way" — clicking a funnel isn't dragging, so the referent is unclear; the original two-sentence form was clearer. Plus Filters is called a "tab" here but a "pane" nine lines below. |
| 4 | Low | microsoft-excel.mdx:155-157, Sheets equivalent |
Not saved and Unsaved in adjacent clauses read as a typo rather than two distinct labels. Worth making the contrast explicit if both are really on screen. |
| 5 | Low | microsoft-excel.mdx:174-183, Sheets equivalent |
Two new UI surfaces (progress footer, overwrite banner) with no {/* TODO: screenshot — ... */} placeholder, which docs-mintlify/CLAUDE.md calls for. "a large workbook takes a while" is also vaguer than it needs to be. |
Not applicable
Security, performance, and test-coverage review are moot — this PR is documentation prose only, with no code, config, or navigation (docs.json) changes. No new pages were added, so no sidebar registration is needed. I did not run the Mintlify dev server; the edits are prose-only within existing MDX blocks, so there's no new syntax that could break the build.
I could not verify the underlying product claims (the Filters drop zone, the tracked Refresh all run, the folder cascade-delete behavior) against the actual add-in or workspace UI — those need confirmation from someone with the shipped build.
- Fix folder content taxonomy: explorations, not reports. - Put the destructive cascade in the Warning callout; ground the permission precondition in per-subfolder permissions instead of an undefined "access to everything inside" notion. - Disambiguate the funnel-button clause and use "Filters pane" (matching the page's existing terminology) instead of "Filters tab". - Make the Not saved / Unsaved contrast explicit instead of reading like a typo. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01GuYqSYDJ9NP1xSsSuFyC7c
|
I'll analyze this and get back to you. |
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01GuYqSYDJ9NP1xSsSuFyC7c
|
Claude finished @vasilev-alex's task in 1m 20s —— View job Docs review (round 2) — all 5 prior findings addressed. 2 low nits remain, both in
|
| # | Prior finding | Status |
|---|---|---|
| 1 | folders.mdx listed reports as folder content |
✅ Fixed — now "subfolders, workbooks, dashboards, and explorations", matching What can go in a folder and sharing.mdx:104 |
| 2 | Emphasis inverted — cascade in prose, permission note in <Warning> |
✅ Fixed — the irreversible cascade is now the <Warning>; the permission precondition demoted to prose |
| 3 | "the same way" ambiguity + "Filters tab" vs "pane" | ✅ Fixed — "or click the funnel buttons to add members to Filters", and Filters pane now matches line 104 |
| 4 | Not saved vs Unsaved read as a typo | ✅ Fixed — the contrast is now explicit (under its sheet vs. a top-level heading when there's no sheet/anchor) |
| 5 | No screenshot placeholder for the new UI surfaces; vague timing hedge | ✅ Fixed — {/* TODO: screenshot — the hand-edit overwrite banner… */} added to both pages, and "takes a while" replaced with "a workbook with many sheets refreshes noticeably slower than a single placement" |
Verification
I normalized the workbook/spreadsheet and add-in/add-on nouns and compared the two integration pages section by section: parity is exact across all seven changes, including the new screenshot placeholder. That's the failure mode these two pages are most prone to, and it's clean.
greyed, **⋮** menu, and the {/* TODO: screenshot — ... */} form all match existing precedent. No new H1s, no <Frame>/component misuse, no legacy "Cube Cloud" naming, no docs.json change needed (no new pages).
Remaining nits (both low)
-
folders.mdx:134-135— inline comment posted. The new prose sentence has a dangling "it" (antecedent now sits inside the<Warning>above), and scoping the block to subfolders may be narrower than the real rule:sharing.mdx:117says direct permissions override inherited ones for any content type, so a workbook or dashboard with its own permissions would presumably block deletion identically. -
Stray mid-paragraph line break —
microsoft-excel.mdx:158-159andgoogle-sheets.mdx:150-151: "…resumes exactly where you left off." ends a short line, then "The pane can also hold…" starts a new one, mid-paragraph. Renders identically in MDX; it just breaks the ~76-col wrap the rest of the file keeps. Purely cosmetic — mentioning rather than commenting inline.
Not applicable
Security, performance, and test-coverage review are moot — documentation prose only, no code or config. I did not run the Mintlify dev server; the edits are prose inside existing MDX blocks plus two MDX comments, so there's no new syntax that could break the build.
I still could not verify the underlying product claims (the Filters drop zone, the tracked Refresh all run, the sheet-at-a-time write ordering, the folder cascade-delete and its permission precondition) against a shipped build — those need confirmation from someone with the product in hand.
… rule Match the direct-permissions-override-inherited rule documented in sharing.mdx: any content type can carry its own permissions, not just subfolders. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01GuYqSYDJ9NP1xSsSuFyC7c
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01GuYqSYDJ9NP1xSsSuFyC7c
|
Claude finished @vasilev-alex's task in 50s —— View job Docs review (round 3) — no new findings. All 6 prior threads resolved and verified fixed; ✅ ready from a docs-review standpoint.Full review
What changed since round 2Two follow-up commits landed:
Both round-2 nits are resolved. No new prose was introduced by either commit beyond those fixes. Verification
Aside (pre-existing, not this PR)
Not applicableSecurity, performance, and test-coverage review are moot — documentation prose only, no code or config. I did not run the Mintlify dev server; the edits are prose inside existing MDX blocks plus two MDX comments, so there's no new syntax that could break the build. The Mintlify preview deployment reports 🟢 Ready. I still could not verify the underlying product claims (the Filters drop zone, the tracked Refresh all run, the sheet-at-a-time write ordering, the Not saved / Unsaved label split, the folder cascade-delete and its permission precondition) against a shipped build — those need confirmation from someone with the product in hand. |

Check List
Description of Changes Made
A batch of documentation updates to the Excel/Google Sheets add-in pages and the folders page, catching up to recently shipped product behavior:
🤖 Generated with Claude Code
https://claude.ai/code/session_01GuYqSYDJ9NP1xSsSuFyC7c
Generated by Claude Code