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

Dbt cloud docs/update enterprise permissions #1805

Closed
wants to merge 49 commits into from

Conversation

matt-winkler
Copy link
Contributor

@matt-winkler matt-winkler commented Jul 27, 2022

Description & motivation

The goal of this PR is to document more specifically which actions users can take based on their assigned permission sets.

dbt Cloud users sometimes encounter challenges with understanding the actions users can take based on their assigned permission sets. In some cases, the current documentation on this page omits key capabilities. For example with the Job Admin permission set, the docs don't state the ability to create dbt Cloud jobs, although a user can in fact create new jobs with this permission set.

image

To-do before merge

Align on approach to communicating this information.

Prerelease docs

If this change is related to functionality in a prerelease version of dbt (delete if not applicable):

jtcohen6 and others added 30 commits May 3, 2021 17:47
…standard-args

Clarify standard args for generic tests
…ilures-new-configs

Tests: store failures, new configs
fix typo - should be `dbt test`
jtcohen6 and others added 18 commits June 16, 2021 12:44
* Edits for on_schema_change

* dbt source freshness

* DBT_ENV_SECRET_ env var

* dbt build first cut

* Redshift profile ra3 property

* Beta callout in migration guide

* Self-review build docs
* state:modified subselectors, modified.macros

* Add build RPC method

* PR feedback
* Switch --models to --select

* BQ snapshot config aliases

* Configurable postgres connect timeout

* Add list --output-keys. Add list RPC method

* Adapter unique_field dbt-labs/dbt-core#3796

* PR feedback: -s replaces -m

* Add BQ execution_project

* Add default property for yaml selectors

* Update migration guide. New fields in sources.json

* Test where config macro

* Dispatch for global macros

* Update build details

* Some self review

* Greedy flag/property for test selection

* Resolve #803 while we're here

* Fix broken link typo
* Very incomplete start of first draft

* Second big pass

* Initial self review. Sidebar reorg

* Continue self review. Address #616

* Add note to migration guide
@netlify
Copy link

netlify bot commented Jul 27, 2022

Deploy Preview for docs-getdbt-com ready!

Name Link
🔨 Latest commit 283f670
🔍 Latest deploy log https://app.netlify.com/sites/docs-getdbt-com/deploys/62e1ddb8afbc210008e1b2eb
😎 Deploy Preview https://deploy-preview-1805--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 settings.

@github-actions github-actions bot added content Improvements or additions to content size: medium This change will take up to a week to address labels Jul 27, 2022
@runleonarun
Copy link
Collaborator

Hey @matt-winkler let me know when this is ready fro review and either @nghi-ly or I will review it!

@matt-winkler
Copy link
Contributor Author

matt-winkler commented Jul 28, 2022

Thanks @runleonarun - are you conceptually on board with updating our documentation along the lines of the example I gave for Job Admin here? Anyone else we'd need to loop in on that decision?

I'm asking a few more questions up front because doing this would illustrate more detail than we've shown historically on this page, so I want to be mindful of the suggested change in approach.

@mirnawong1 mirnawong1 added the merge-conflict-cli This PR has a merge conflict that needs to be resolved label Nov 30, 2022
@matthewshaver
Copy link
Contributor

Closing due to age - we have since redesigned the permissions page to more accurately display what each role has access to https://docs.getdbt.com/docs/cloud/manage-access/enterprise-permissions

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 merge-conflict-cli This PR has a merge conflict that needs to be resolved size: medium This change will take up to a week to address
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants