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
Merged
merged 3 commits into from
Jul 3, 2025
Merged

Desktop: rm self-diagnose tool #22900

merged 3 commits into from
Jul 3, 2025

Conversation

@aevesdocker
Copy link
Contributor

@aevesdocker aevesdocker commented Jun 24, 2025

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

aevesdocker commented Jun 24, 2025

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

LGTM!

@aevesdocker aevesdocker merged commit d22f7c2 into docker:main Jul 3, 2025
15 checks passed
sarahsanders-docker pushed a commit to sarahsanders-docker/docs that referenced this pull request Jul 10, 2025
@aevesdocker @sarahsanders-docker
Desktop: rm self-diagnose tool (docker#22900)
dc19b6b 
<!--Delete sections as needed -->

## Description

Self-diagnose tool has been deprecated

## Related issues or tickets

docker/pinata#34924

## Reviews

<!-- Notes for reviewers here -->
<!-- List applicable reviews (optionally @tag reviewers) -->

- [ ] Technical review
- [ ] Editorial review
- [ ] Product review
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers

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

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

Reviewers whose approvals may not affect merge requirements

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