Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

clarify SL set up permissions #3950

Merged
merged 10 commits into from
Aug 18, 2023
Merged

clarify SL set up permissions #3950

merged 10 commits into from
Aug 18, 2023

Conversation

mirnawong1
Copy link
Contributor

this pr clarifies the permissions needed to configure the dbt SL per slack thread

@mirnawong1 mirnawong1 requested a review from a team as a code owner August 18, 2023 19:02
@netlify
Copy link

netlify bot commented Aug 18, 2023

Deploy Preview for docs-getdbt-com ready!

Name Link
🔨 Latest commit fe7b5da
🔍 Latest deploy log https://app.netlify.com/sites/docs-getdbt-com/deploys/64dfd62c836c88000833e1be
😎 Deploy Preview https://deploy-preview-3950--docs-getdbt-com.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@github-actions github-actions bot added content Improvements or additions to content Docs team Authored by the Docs team @dbt Labs size: x-small This change will take under 3 hours to fix. labels Aug 18, 2023
Copy link
Contributor Author

@mirnawong1 mirnawong1 left a comment

Choose a reason for hiding this comment

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

clarify

website/snippets/_new-sl-setup.md Outdated Show resolved Hide resolved
Copy link
Contributor Author

@mirnawong1 mirnawong1 left a comment

Choose a reason for hiding this comment

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

More clarification

website/snippets/_new-sl-setup.md Outdated Show resolved Hide resolved
Copy link
Contributor Author

@mirnawong1 mirnawong1 left a comment

Choose a reason for hiding this comment

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

Add

website/snippets/_new-sl-setup.md Outdated Show resolved Hide resolved
Copy link
Contributor Author

@mirnawong1 mirnawong1 left a comment

Choose a reason for hiding this comment

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

clarify

website/snippets/_new-sl-setup.md Outdated Show resolved Hide resolved
website/snippets/_new-sl-setup.md Outdated Show resolved Hide resolved
Copy link
Contributor Author

@mirnawong1 mirnawong1 left a comment

Choose a reason for hiding this comment

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

Add admin

website/snippets/_new-sl-setup.md Outdated Show resolved Hide resolved
Copy link
Contributor

@rpourzand rpourzand left a comment

Choose a reason for hiding this comment

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

Changes noted in comments

- You must have a dbt Cloud Team or Enterprise [multi-tenant](/docs/cloud/about-cloud/regions-ip-addresses) deployment, hosted in North America.
- You must be part of the Owner group, and have the correct [license](/docs/cloud/manage-access/seats-and-users) and [permissions](/docs/cloud/manage-access/self-service-permissions) to configure the Semantic Layer:
Copy link
Contributor

Choose a reason for hiding this comment

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

I think the only thing to change here is that if you are an Account Admin, you actually don't have to be an Owner. You can just be "Everyone" in this case

Copy link
Contributor Author

@mirnawong1 mirnawong1 left a comment

Choose a reason for hiding this comment

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

clarify acct admin

website/snippets/_new-sl-setup.md Outdated Show resolved Hide resolved
Copy link
Contributor

@rpourzand rpourzand left a comment

Choose a reason for hiding this comment

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

lgtm!

@mirnawong1 mirnawong1 merged commit cf76cbf into current Aug 18, 2023
@mirnawong1 mirnawong1 deleted the mwong-sl-clarify-perms branch August 18, 2023 20:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content Improvements or additions to content Docs team Authored by the Docs team @dbt Labs size: x-small This change will take under 3 hours to fix.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants