Skip to content

docs: tableau sls least privilege mode - #11063

Merged
MikeNitsenko merged 1 commit into
masterfrom
docs-tableau-sls-least-privilege-mode
Jun 11, 2026
Merged

docs: tableau sls least privilege mode#11063
MikeNitsenko merged 1 commit into
masterfrom
docs-tableau-sls-least-privilege-mode

Conversation

@MikeNitsenko

Copy link
Copy Markdown
Contributor

No description provided.

@MikeNitsenko
MikeNitsenko requested a review from keydunov as a code owner June 11, 2026 09:48
@github-actions github-actions Bot added the pr:community Contribution from Cube.js community members. label Jun 11, 2026
@claude

claude Bot commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Claude finished @MikeNitsenko's task in 32s —— View job


Review

Docs-only PR adding a Least-privilege mode section to the Tableau Semantic Layer Sync page. The change is clean and follows the repo conventions in docs-mintlify/CLAUDE.md.

Verdict: LGTM — 0 high, 0 medium, 2 low (nits)

What's good

  • Correct Mintlify components (<CodeGroup>, <Note>, <Info>, <Tip>) and consistent with the rest of the page.
  • Heading level (### under ## Configuration) is right — no H1 in the body.
  • Both Python and JavaScript examples mirror the style used earlier in the file.
  • The behavioral consequences of the flag (userNamesWithAccess / groupNamesWithAccess ignored, project pre-creation requirement) are spelled out clearly.

Low / nits

  1. docs-mintlify/docs/integrations/semantic-layer-sync/tableau.mdx:188-202 — the <CodeGroup> examples are partial snippets (just the config block) while every other example on the page is a full semantic_layer_sync / semanticLayerSync config. A reader copy-pasting might be briefly confused about where leastPrivilegeMode slots in. Consider either:

    • keeping the partial snippet but tagging the fences with something like title="Python (snippet)", or
    • showing the field inline within a complete example as done in the Tableau Cloud / Tableau Server sections above.
  2. docs-mintlify/docs/integrations/semantic-layer-sync/tableau.mdx:170 — the section currently lives at the bottom of ## Configuration, after the Tableau Server example. Since it applies to both Tableau Cloud and Tableau Server, that placement is fine, but a one-line lead-in like "Applies to Tableau Cloud and Tableau Server." would remove any ambiguity (Tableau Desktop has no API-based sync, so the flag obviously doesn't apply there).

Nothing blocking. Security / performance / testing sections are N/A for a docs-only change.

• Branch: `docs-tableau-sls-least-privilege-mode`

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds documentation for a new Least-privilege mode for Tableau Semantic Layer Sync, explaining how to run syncs without requiring a Site Administrator PAT by disabling Cube-managed Tableau project permission management.

Changes:

  • Documented what “Least-privilege mode” does and its behavioral implications (permissions management and project pre-existence requirements).
  • Documented how to enable the mode via the sync wizard UI and via configuration (leastPrivilegeMode).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs-mintlify/docs/integrations/semantic-layer-sync/tableau.mdx
Comment thread docs-mintlify/docs/integrations/semantic-layer-sync/tableau.mdx
@mintlify

mintlify Bot commented Jun 11, 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 Jun 11, 2026, 9:53 AM

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

@MikeNitsenko
MikeNitsenko merged commit 3b8b61e into master Jun 11, 2026
15 checks passed
@MikeNitsenko
MikeNitsenko deleted the docs-tableau-sls-least-privilege-mode branch June 11, 2026 16:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:community Contribution from Cube.js community members.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants