-
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
Editing role #6460
Editing role #6460
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@@ -56,7 +56,7 @@ If there has been a change in user roles or Slack permissions where you no longe | |||
::: | |||
|
|||
### Prerequisites | |||
- You must be an administrator of the Slack workspace. | |||
- You must be a Slack Workspace Owner. |
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.
Hiya @matthewshaver
This is more of a question - in the docs "workspace" isn't capitalised and "owner" is capitalised so should "Slack Workspace Owner" be "Slack workspace Owner"?
Kind Regards
Natalie
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.
Great question! We treat it the same way that Slack treats it, rather than porting it over to our style guide.
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.
Hiya @matthewshaver
I've approved this for you but left a QQ as I wasn't sure about the punctuation after reading our docs. It's a non-blocker.
Kind Regards
Natalie
What are you changing in this pull request and why?
Since the dbt Cloud app is not in the official Slack marketplace only Workspace Owners can install
Checklist
🚀 Deployment available! Here are the direct links to the updated files: