Skip to content

Conversation

@sarahsanders-docker
Copy link
Contributor

@sarahsanders-docker sarahsanders-docker commented Oct 1, 2024

Description

Subscription docs needed updated to replace old login flow (Docker Hub) with new login flow (Docker Home). The Docker Home flow is:

  1. Docker Home login (app.docker.home)
  2. Click avatar
  3. Select Billing from drop-down

Related issues or tickets

ENGDOCS-2236

Reviews

  • Editorial review

@netlify
Copy link

netlify bot commented Oct 1, 2024

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit f89dd20
🔍 Latest deploy log https://app.netlify.com/sites/docsdocker/deploys/67040a43ff2f5800089e54a4
😎 Deploy Preview https://deploy-preview-21044--docsdocker.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 configuration.

@github-actions github-actions bot added the area/subscription Relates to Docker pricing and subscription label Oct 1, 2024
Copy link
Contributor

@craig-osterhout craig-osterhout left a comment

Choose a reason for hiding this comment

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

When verifying, a few UI wording inconsistencies popped up that weren't in the original request. Sorry, I didn't realize there were that many issues with the existing content when passing this to you. I'll ping you on slack on how to set up a demo org on staging so you can go through the flow.

2. Optional. If you're upgrading from a free user account to a Team subscription and want to keep your account name, [convert your user account into an organization](../../admin/convert-account.md).

3. Select your avatar in the top-right corner of Docker Hub.
3. Select your **avatar** to expand the drop-down menu.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggest not bolding avatar and only bolding UI text. This applies to the other occurrences too. Not a hard-and-fast rule though.

Co-authored-by: Craig Osterhout <103533812+craig-osterhout@users.noreply.github.com>
Copy link
Contributor

@craig-osterhout craig-osterhout left a comment

Choose a reason for hiding this comment

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

LGTM

@sarahsanders-docker sarahsanders-docker merged commit a6761d5 into docker:main Oct 7, 2024
14 checks passed
@sarahsanders-docker sarahsanders-docker deleted the ENGDOCS-2236 branch October 7, 2024 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/subscription Relates to Docker pricing and subscription status/review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants