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 Snowflake account location #1762

Merged
merged 2 commits into from
Aug 4, 2022
Merged

Conversation

lcochran400
Copy link
Contributor

Because the url of Snowflake, where the account information can be easily found, changes based on which version of Snowflake the user is on (Classic Console vs AppUI/Snowsite), it can be confusing to identify the location of the account information. I added a distinction for each console to provide more clarity.

Description & motivation

To-do before merge

Prerelease docs

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

Checklist

If you added new pages (delete if not applicable):

  • The page has been added to website/sidebars.js
  • The new page has a unique filename

If you removed existing pages (delete if not applicable):

  • The page has been removed from website/sidebars.js
  • An entry has been added to _redirects

Because the url of Snowflake, where the account information can be easily found, changes based on which version of Snowflake the user is on (Classic Console vs AppUI/Snowsite), it can be confusing to identify the location of the account information. I added a distinction for each console to provide more clarity.
@lcochran400 lcochran400 requested a review from runleonarun as a code owner July 21, 2022 17:15
@netlify
Copy link

netlify bot commented Jul 21, 2022

Deploy Preview for docs-getdbt-com ready!

Name Link
🔨 Latest commit 1512f48
🔍 Latest deploy log https://app.netlify.com/sites/docs-getdbt-com/deploys/62ea8dd98cafff000815fb51
😎 Deploy Preview https://deploy-preview-1762--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: x-small This change will take under 3 hours to fix. labels Jul 21, 2022
Copy link
Contributor

@coapacetic coapacetic left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@@ -242,7 +242,7 @@ Now let's formally set up the connection between dbt Cloud and Snowflake.

1. Click on the "Snowflake" icon to set up your connection.
2. Enter the following information under Snowflake settings.
* **Account:** Your account is going to be the url of your Snowflake trial account up to the period before snowflakecomputing.com. So, if this is the full url of my trial account: `oq65696.west-us-2.azure.snowflakecomputing.com` then the account will be: `oq65696.west-us-2.azure`. You can read more about Snowflake account identifiers [here](https://docs.snowflake.com/en/user-guide/admin-account-identifier.html).
* **Account:** Your account is located in the url of your Snowflake trial account, excluding the `snowflakecomputing.com`. The location of your account information will vary based on which version of Snowflake you are using. For Snowflake's Classic Console, your url may look like this: `oq65696.west-us-2.azure.snowflakecomputing.com`; if you're using the AppUI, or Snowsite, then the url may look like this: `snowflakecomputing.com/west-us-2.azure/oq65696`. In both of these instances, your account will be: `oq65696.west-us-2.azure`. You can read more about Snowflake account identifiers [here](https://docs.snowflake.com/en/user-guide/admin-account-identifier.html).
Copy link
Contributor

Choose a reason for hiding this comment

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

It is actually Snowsight, not Snowsite

Copy link
Collaborator

@runleonarun runleonarun left a comment

Choose a reason for hiding this comment

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

Thank you for opening this PR @lcochran-dbt! I slightly reworded to make the sentence active. We've been trying to standardize a few things like references to more info. Not everything has made it to the style guide just yet.

@lcochran400 lcochran400 requested a review from annafil as a code owner August 3, 2022 15:01
@lcochran400 lcochran400 merged commit b9e27a8 into current Aug 4, 2022
@lcochran400 lcochran400 deleted the lcochran-dbt-snowflake-acct branch August 4, 2022 15:17
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 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.

4 participants