cdp-ucsc / doc

0 stars 0 forks source link

48 Onboarding documentation refresh #47

Closed insuhpak closed 1 month ago

tgraham-ucsc commented 11 months ago

Hi Insuh, nice work on the docs!!

I've made some edits/suggestions in a branch that was created from this one. You can look at the changes I made at the link below or pull, checkout and build from the branch tg-46-doc-review to see changes.

https://github.com/cdp-ucsc/doc/compare/46-doc-review-workstation-set-up-instructions-macos-specific...tg-46-doc-review

Still working on it, but wanted to get your thoughts sooner rather than later

Thanks!

insuhpak commented 11 months ago

@tgraham-ucsc I think somewhere we should specify that in order to successfully set up dbt Cloud the user must have requested and been granted access to both the GitHub organization and Snowflake. I think someone setting up Cloud could think that these are all separate so they only request access to GitHub and not Snowflake. Or just dbt Cloud. And that would slow down the process for them.

I know you removed the section from the dbt cloud document. Maybe it fits better somewhere else?

Other than that your changes look good! And the dbt Core documentation update will be coming.

Also, I was wondering when users get approved for Snowflake will they receive all of the credential information that is needed to connect to Snowflake in Cloud?

Similarly, is there somewhere in the request access process tha they are informed of the specific projects (and their repos) they have access to? This is relevant for dbt cloud setup since they need to connect to Snowflake for each project they will be working on.

insuhpak commented 1 month ago

Lapsed issue. Closing due to clean-up.