Skip to content
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.

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

Remove obsolete fields from metadata.json #116

Merged
merged 3 commits into from
Apr 4, 2022
Merged

Remove obsolete fields from metadata.json #116

merged 3 commits into from
Apr 4, 2022

Conversation

gtardif
Copy link
Contributor

@gtardif gtardif commented Mar 31, 2022

No description provided.

@@ -16,9 +16,6 @@ The metadata.json file must follow the format :

```json
{
"name": "volumes-share",
"provider": "Docker Inc.",
"icon": "extension-icon.svg",
Copy link

Choose a reason for hiding this comment

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

seems to me the icon is needed still, or else, the extension uses a default icon. I kinda expecting the com.docker.desktop.extension.icon docker label would be used instead, but apparently not.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah, thx for noticing! Indeed the name and provider metadata are provided with the image label. Icon appearing in the Marketplace is also provided with an image label (for extensions in the marketplace), but icon displayed in the left-hand-side drawer are available offline and are still extracted from the extension image itself.

@@ -65,7 +65,8 @@ FROM alpine:3.15
LABEL org.opencontainers.image.title="HelloBackend" \
org.opencontainers.image.description="A sample extension that runs a shell script inside a container's Desktop VM." \
org.opencontainers.image.vendor="Docker Inc." \
com.docker.desktop.extension.api.version="1.0.0-beta.1"
com.docker.desktop.extension.api.version="1.0.0-beta.1" \
com.docker.desktop.extension.icon="https://www.docker.com/sites/default/files/d8/2019-07/Moby-logo.png"
Copy link
Contributor

Choose a reason for hiding this comment

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

@gtardif 404 Not Found?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah thx to notice this. Updated to link from our media-resources.

Signed-off-by: Guillaume Tardif <[email protected]>
@gtardif gtardif requested a review from felipecruz91 April 4, 2022 13:44
@gtardif gtardif merged commit 2895bb2 into docker:main Apr 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants