-
Notifications
You must be signed in to change notification settings - Fork 5
[NCl-9058] Build Push and Milestone close #641
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
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.
Verified on my local dev env with BC: TC30-temporary-push
, brew-push list component and details page looks all good and no error or warning related to this change.
The new "Close Milestone" modal looks good, and I was able to close a milestone with Skip Brew push
enabled, but I can't close it without Skip Brew push
option. Not sure if it is due to dev env endpoint issue? @patrikk0123 do you have any idea on that?
I also tried to rebase NCL-9126 PR(https://github.com/project-ncl/pnc-web-ui-react/pull/638/files) on this code base, everything looks good as well. |
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.
Other than the issue raised above, it looks good to me
@DnsZhou End date will be set only after all brew pushes have finished successfully |
5d5ed2a
to
0f40af2
Compare
…ers and attributes
No description provided.