-
Notifications
You must be signed in to change notification settings - Fork 960
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
Add more warehouses to database permissions #3943
Add more warehouses to database permissions #3943
Conversation
Hello!👋 Thanks for contributing to the dbt product documentation and opening this pull request! ✨ |
✅ Deploy Preview for docs-getdbt-com ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Hey @mirnawong1 is there anything we need to change to be able to get this merged please? |
website/docs/reference/database-permissions/databricks-permissions.md
Outdated
Show resolved
Hide resolved
@mirnawong1 is attempting to deploy a commit to the dbt-labs Team on Vercel. A member of the Team first needs to authorize it. |
this is so great @agnessnowplow and @rlh1994 ! thanks so much for opening this up and providing such diligent and detailed contributions 😍 ! apologies for the delay here and I'll review this right now - am going to loop in @dataders as well for his technical help - thank you for flagging @rlh1994 ! |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
website/docs/reference/database-permissions/redshift-permissions.md
Outdated
Show resolved
Hide resolved
website/docs/reference/database-permissions/postgres-permissions.md
Outdated
Show resolved
Hide resolved
adding discourse article to resolves issue dbt-labs#3824
website/docs/reference/database-permissions/snowflake-permissions.md
Outdated
Show resolved
Hide resolved
website/docs/reference/database-permissions/snowflake-permissions.md
Outdated
Show resolved
Hide resolved
website/docs/reference/database-permissions/about-database-permissions.md
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hey @agnessnowplow and @rlh1994 ! i think we're at a good place now and the deploy preview looks pretty great! i really appreciate you opening this wonderful pr and flagging this for us - this page needed some attention and thanks to your contribution, it now has much more info.
I'll get this merged for you but let me know if you have any questions or concerns
What are you changing in this pull request and why?
The proposed changes are regarding this issue raised last week: #3901
It would be nice to have more warehouses within References / warehouse permissions with sample grants, this PR aims to follow the already existing Snowflake example and add Databricks, Redshift, and Postgres as well (skipping BigQuery due to their unique IAM access control management)
#Resolves #3824
Checklist
Adding new pages (delete if not applicable):
Add page to
website/sidebars.js
Provide a unique filename for the new page
Needs to have a technical review, at least to verify the proposed changes are correct and possibly expand with other examples.
Tested it locally (npm run)