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

update README and add new screenshots #33

Conversation

vrothberg
Copy link
Collaborator

Update the README with instructions on how to install and use the extension along with some fresh screenshots.

@vrothberg vrothberg requested a review from a team as a code owner February 9, 2024 10:42
@vrothberg vrothberg requested review from dgolovin and removed request for a team February 9, 2024 10:42
@vrothberg
Copy link
Collaborator Author

@jeffmaury @dgolovin @benoitf PTAL

Copy link
Collaborator

@benoitf benoitf left a comment

Choose a reason for hiding this comment

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

I reported few suggestion glitches

FYI README content is usually included in the catalog and then it's being displayed when you open special links to podman desktop

using the shortcut ![image](file.png) won't work as it's relative links

you should update it with links starting with https://raw.githubusercontent.com/redhat-developer/podman-desktop-redhat-account-ext/main/images/...

README.md Outdated Show resolved Hide resolved

The first available Alpha image is `quay.io/redhat-developer/podman-desktop-redhat-account-ext:0.0.1-alpha.1`.
Copy link
Collaborator

Choose a reason for hiding this comment

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

will it be published in the catalog ?
https://github.com/containers/podman-desktop-catalog/tree/gh-pages

about the versioning
there is a version named 0.0.1 there so it's strange to have 0.0.1 being older than 0.0.1-alpha1

then you could have links like : podman-desktop:extension/redhat.redhat-authentication that will prompt you if you have podman desktop installed to install this extension

Copy link
Collaborator

Choose a reason for hiding this comment

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

I will change it to 0.0.2-alpha.1

Copy link
Collaborator

Choose a reason for hiding this comment

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

In a follow up PR

README.md Outdated Show resolved Hide resolved
Update the README with instructions on how to install and use the
extension along with some fresh screenshots.

Signed-off-by: Valentin Rothberg <vrothberg@redhat.com>
@vrothberg
Copy link
Collaborator Author

you should update it with links starting with https://raw.githubusercontent.com/redhat-developer/podman-desktop-redhat-account-ext/main/images/...

Thanks! Can we do that in a follow-up PR? Otherwise, they won't show up now for review as the images are not yet uploaded/pushed. Or is there a way to do that?

@benoitf
Copy link
Collaborator

benoitf commented Feb 9, 2024

follow-up works for me, I just wanted to bring awareness and approved the PR

@vrothberg
Copy link
Collaborator Author

Thanks for the review, @benoitf !

@vrothberg vrothberg force-pushed the hackathon-subscription-vrothberg branch from dfcb118 to 6dafa8c Compare February 9, 2024 12:10
@vrothberg
Copy link
Collaborator Author

@rhatdan FYI

@dgolovin
Copy link
Collaborator

dgolovin commented Feb 9, 2024

I'll take care of images in follow up PR as well

@dgolovin dgolovin merged commit 9a18795 into redhat-developer:0.0.1-alpha.x Feb 9, 2024
3 checks passed
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