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

update billing screenshot #3502

Merged
merged 10 commits into from
Jun 13, 2023
Merged

update billing screenshot #3502

merged 10 commits into from
Jun 13, 2023

Conversation

mirnawong1
Copy link
Contributor

updating billing screenshot and unifying them per pr #3497 where user was confused with outdated screenshots since there was a ui change.

Screenshots updated in https://docs.getdbt.com/docs/cloud/manage-access/seats-and-users#licenses and https://docs.getdbt.com/faqs/Accounts/delete-users pages

cc @Lana-hl

@mirnawong1 mirnawong1 requested a review from a team as a code owner June 9, 2023 15:05
@github-actions github-actions bot added the content Improvements or additions to content label Jun 9, 2023
@netlify
Copy link

netlify bot commented Jun 9, 2023

Deploy Preview for docs-getdbt-com ready!

Name Link
🔨 Latest commit 61f4ee1
🔍 Latest deploy log https://app.netlify.com/sites/docs-getdbt-com/deploys/64884258ea434e0009f86df7
😎 Deploy Preview https://deploy-preview-3502--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 Docs team Authored by the Docs team @dbt Labs size: small This change will take 1 to 2 days to address labels Jun 9, 2023
@mirnawong1 mirnawong1 enabled auto-merge June 9, 2023 15:07
@mirnawong1 mirnawong1 disabled auto-merge June 9, 2023 15:07
Copy link
Contributor

@nghi-ly nghi-ly left a comment

Choose a reason for hiding this comment

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

these are great updates, @mirnawong1 ! thank you!

website/docs/faqs/Accounts/delete-users.md Outdated Show resolved Hide resolved

If you are on a **Teams** plan and you are deleting users to reduce the number of billable seats, you also need to take these steps to lower the license count:
1. In **Account Settings**, select **Billing**.
2. Enter the number of developer seats you want and click **Save**.
2. Enter the number of developer seats you want and make sure you fill in all the payment details, including the **Billing Address** section. Leaving these blank won't allow you to save your changes.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
2. Enter the number of developer seats you want and make sure you fill in all the payment details, including the **Billing Address** section. Leaving these blank won't allow you to save your changes.
2. Enter the number of developer seats you want and make sure you fill in all the payment details, including the **Billing Address** section. If any field is left blank, you won't be able to save your changes.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

this sugg is passive voice so i'm switching it to active


<Lightbox src="/img/docs/dbt-cloud/change-developer-seats.png" title="Developer seats" />
Copy link
Contributor

Choose a reason for hiding this comment

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

is this PNG being used in other pages? if not, suggest deleting it to keep our /img folder clean

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Screenshot 2023-06-12 at 13 52 10

@@ -10,19 +10,20 @@ To delete a user in dbt Cloud, you must be an account owner or have admin privil

1. From dbt Cloud, click the gear icon at the top right and select **Account Settings**.

<Lightbox src="/img/docs/dbt-cloud/Navigate To Account Settings.png" title="Navigate to account settings" />
<Lightbox src="/img/docs/dbt-cloud/Navigate To Account Settings.png" width="85%" title="Navigate to account settings" />
Copy link
Contributor

Choose a reason for hiding this comment

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

note: same thought as before regarding figure captions. not flagging/not adding a suggestion since you only updated the size on this line and not the text

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok yea i just wanted to change the sizing because it was too small. regarding the title - it looks like this was inherited from someone's previous pr that we all touched at some point.

@mirnawong1
Copy link
Contributor Author

thank you @nghi-ly , can you give this another review? ✨

@mirnawong1 mirnawong1 enabled auto-merge June 12, 2023 13:02
@mirnawong1
Copy link
Contributor Author

thank @nghi-ly !

@mirnawong1 mirnawong1 merged commit 12b60d2 into current Jun 13, 2023
@mirnawong1 mirnawong1 deleted the mwong-billing-screenshot branch June 13, 2023 10:22
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 Docs team Authored by the Docs team @dbt Labs size: small This change will take 1 to 2 days to address
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants