Skip to content

Conversation

@craig-osterhout
Copy link
Contributor

Description

Removed company management from Hub.

Related issues or tickets

ENGDOCS-2240

Reviews

  • Technical review
  • Editorial review

Signed-off-by: Craig <craig.osterhout@docker.com>
@netlify
Copy link

netlify bot commented Sep 30, 2024

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 45873dc
🔍 Latest deploy log https://app.netlify.com/sites/docsdocker/deploys/67042346b993780008c6b371
😎 Deploy Preview https://deploy-preview-21035--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 area/hub Issue affects Docker Hub area/security area/admin Relates to Docker Admin hugo Updates related to hugo labels Sep 30, 2024
aevesdocker
aevesdocker previously approved these changes Oct 1, 2024
Copy link
Contributor

@aevesdocker aevesdocker left a comment

Choose a reason for hiding this comment

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

LGTM

@craig-osterhout craig-osterhout added the status/do-not-merge Pull requests that are awaiting some event or decision before they can be merged. label Oct 1, 2024
@chrstnfrrs
Copy link
Contributor

Screenshot 2024-10-07 at 10 37 45 AM

Should we change this sentence to provide greater accuracy?

but the Admin Console includes enhanced features for company-level management.

I think something like, "but the Admin Console includes company-level management and enhanced features for organization management", might be more accurate?

@chrstnfrrs
Copy link
Contributor

Question: On the main Company Management page, should we have a call to action or alert that notifies users that this is only in admin console?

Screenshot 2024-10-07 at 10 40 13 AM

Signed-off-by: Craig <craig.osterhout@docker.com>
@craig-osterhout craig-osterhout requested a review from a team October 7, 2024 18:08
@craig-osterhout craig-osterhout removed the status/do-not-merge Pull requests that are awaiting some event or decision before they can be merged. label Oct 7, 2024
It's the world’s largest repository of container images with an array of content sources including container community developers, open source projects, and independent software vendors (ISV) building and distributing their code in containers.

Docker Hub is also where you can go to [change your Docker account settings and carry out administrative tasks](/admin/). If you have a Docker Team or Business subscription, you can also carry out administrative tasks in the [Docker Admin Console](https://admin.docker.com).
Docker Hub is also where you can go to [carry out administrative tasks for organizations](/admin/). If you have a Docker Team or Business subscription, you can also carry out administrative tasks in the [Docker Admin Console](https://admin.docker.com).
Copy link
Contributor

Choose a reason for hiding this comment

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

Just a question, not a blocker: is the Docker Admin Console URL https://admin.docker.com or app.docker.com/admin?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good catch. It should be app.docker.com/admin but the old URL, admin.docker.com will redirect to it. Capturing this as a follow-up in ENGDOCS-2246 to get this out asap and because there's probably more than one wrong occurrence.

Copy link
Contributor

@sarahsanders-docker sarahsanders-docker left a comment

Choose a reason for hiding this comment

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

Left one comment about use of admin.docker.com vs. app.docker.com but the rest LGTM

@craig-osterhout craig-osterhout merged commit 3bb24b6 into docker:main Oct 7, 2024
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/admin Relates to Docker Admin area/hub Issue affects Docker Hub area/security hugo Updates related to hugo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants