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

Click the "DEPLOY" button directly will fail if the repository contain no image whose tag is latest #4788

Closed
ywk253100 opened this issue Apr 26, 2018 · 8 comments

Comments

@ywk253100
Copy link
Contributor

If the repository library/hello-world contains no image whose tag is latest, clicking the DEPLOY buttion will fail as the latest is the default value if no tag is specified:
deploy
Users can click the Add addtional info and choose an existing tag to work around this issue:
deploy_02

@lweitzman
Copy link

we should follow the clarity standards and have the main action "Deploy" along with the menu "Actions v". in the menu we should have "Deploy with options" and a dividing line and "Delete"
when there are no images with the "latest" tag, then the Deploy button should be grayed out and not removed. see https://vmware.invisionapp.com/share/ZVF75AZAW#/271907734_Projects_-_Repositories_-_Card_V1-4
where you can click on the menu for the wordpress repo.
screen shot 2018-04-26 at 9 16 23 am
issue #4787 is related

@steven-zou
Copy link
Contributor

@lweitzman The button style should be kept consistent with the button in Admiral side

@ninjadq
Copy link
Member

ninjadq commented May 2, 2018

@lweitzman I agree with you, but Admiral does not strictly obey the clarity standard. Most of the engineering work of your design is on Admiral side. But currently, we should follow their style, or can you persuade them?

@lweitzman
Copy link

@ninjadq asking PM and eng leads about consistency of menus with the hope that i can persuade them

@reasonerjt
Copy link
Contributor

Before we reach agreement with Admiral.
On Harbor's UI we can mitigate the issue by:
When there's no "latest" tag of the repo, the deploy button will be disabled and a tooltip will be shown to ask user set the "additional info"

@reasonerjt reasonerjt added this to the Sprint 34 milestone May 8, 2018
@lweitzman
Copy link

Instead of "additional info" i recommend that we rename that option to "Deploy with options" as stated above. I think its much clearer as to what the action is going to do.

@ninjadq ninjadq changed the title Click the "DEPLOY" button directly will fail if the repository contain no image whose tag is latest Click the "DEPLOY" button will emit add info event if the repository contain no image whose tag is latest May 14, 2018
@ninjadq ninjadq changed the title Click the "DEPLOY" button will emit add info event if the repository contain no image whose tag is latest Click the "DEPLOY" button directly will fail if the repository contain no image whose tag is latest May 14, 2018
@stuclem
Copy link
Contributor

stuclem commented May 15, 2018

Proposed release note:

  • Clicking Deploy for an image in the Internal Repositories view fails if no latest tag is present. #4788This will be fixed in an update release.

@reasonerjt is this OK?

@stuclem stuclem removed the kind/note label May 16, 2018
@reasonerjt reasonerjt modified the milestones: Sprint 34, Sprint 35 May 23, 2018
@reasonerjt
Copy link
Contributor

We'll use this issue to track the text change, will close it once we confirm the text in the label and make necessary change.

@ninjadq ninjadq closed this as completed May 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants