Skip to content

Conversation

@aevesdocker
Copy link
Contributor

Description

Related issues or tickets

Reviews

  • Technical review
  • Editorial review
  • Product review

@github-actions github-actions bot added the area/desktop Issue affects a desktop edition of Docker. E.g docker for mac label Jan 9, 2025
@netlify
Copy link

netlify bot commented Jan 9, 2025

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 764d15e
🔍 Latest deploy log https://app.netlify.com/sites/docsdocker/deploys/67804241c9900a0008ceadbb
😎 Deploy Preview https://deploy-preview-21769--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.

@aevesdocker aevesdocker marked this pull request as ready for review January 9, 2025 21:19
cdupuis
cdupuis previously requested changes Jan 9, 2025
Copy link
Contributor

@cdupuis cdupuis left a comment

Choose a reason for hiding this comment

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

Do not merge.

Copy link
Contributor

@gtardif gtardif left a comment

Choose a reason for hiding this comment

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

Overall LGTM, some minor comment/suggestion


You should now see the Docker Desktop Dashboard.

### Install a patch if you have version 4.34 - 4.36
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
### Install a patch if you have version 4.34 - 4.36
### Install a patch if you want to continue using Desktop version 4.34 - 4.36

Maybe, to make it clearer that the recommended option is to update to 4.37.2?


### Wait for a patch for versions 4.32 - 4.33

For versions 4.32 - 4.33, a patch fix is in progress. If you need an immediate solution, you can use the following workaround:
Copy link
Contributor

Choose a reason for hiding this comment

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

If we communicate this tomorrow, we might have the patch release already available, that would make the message simpler

@aevesdocker aevesdocker merged commit 4d160e7 into docker:main Jan 9, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/desktop Issue affects a desktop edition of Docker. E.g docker for mac

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants