-
Notifications
You must be signed in to change notification settings - Fork 232
Update overview to use toast notifications #1654
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
Conversation
[test] |
message: _.capitalize(buildType) + " " + buildName + " successfully created.", | ||
links: [{ | ||
href: buildURL, | ||
label: 'View ' + _.capitalize(buildType) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
any reason to not have the View Log link here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The reason I linked to the build instead of the log is because the build might not start immediately. But I'm not against changing it.
rebased |
Evaluated for origin web console test up to eab985c |
[merge] |
Evaluated for origin web console merge up to eab985c |
Origin Web Console Test Results: SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin_web_console/1474/) (Base Commit: 9ca02da) |
Origin Web Console Merge Results: SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin_web_console/1476/) (Base Commit: 09106a3) |
https://trello.com/c/6WVJdgjL