Skip to content

Desktop: rm self-diagnose tool #22900

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Desktop: rm self-diagnose tool #22900

wants to merge 3 commits into from

Conversation

aevesdocker
Copy link
Contributor

Description

Self-diagnose tool has been deprecated

Related issues or tickets

https://github.com/docker/pinata/pull/34924

Reviews

  • Technical review
  • Editorial review
  • Product review

@netlify Netlify
Copy link

netlify bot commented Jun 24, 2025 •
edited
Loading

✅ Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 46b5062
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/685a97df50c62b0008530638
😎 Deploy Preview https://deploy-preview-22900--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 project configuration.

@github-actions github-actions bot added the area/desktop Issue affects a desktop edition of Docker. E.g docker for mac label Jun 24, 2025
@aevesdocker aevesdocker added this to the 4.43 milestone Jun 24, 2025
@karman-docker
Copy link
Contributor

karman-docker commented Jun 24, 2025 •
edited
Loading

Hi @aevesdocker thanks for this PR,
Just a thought, since we removed that link on the DD dashboard in 4.43, the link will still be displayed on older version of DD. So I was wondering whether we can still keep the "Self-diagnose tool" section in the docs, but replace its contents with "This tool has now been deprecated."

@aevesdocker
Copy link
Contributor Author

Oh that's true @karman-docker, we don't want to break links. Have amended!

@aevesdocker aevesdocker requested a review from a team June 24, 2025 12:29
sarahsanders-docker
sarahsanders-docker approved these changes Jun 24, 2025
View reviewed changes
Copy link
Collaborator

@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.

LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@karman-docker karman-docker karman-docker approved these changes

@sarahsanders-docker sarahsanders-docker sarahsanders-docker approved these changes

Assignees
No one assigned
Labels
area/desktop Issue affects a desktop edition of Docker. E.g docker for mac status/review
Projects
None yet
Milestone
4.43
Development

Successfully merging this pull request may close these issues.
3 participants
@aevesdocker @karman-docker @sarahsanders-docker