-
Notifications
You must be signed in to change notification settings - Fork 189
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
Fix #2762: Indicate better build information in build-header #2907
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.
I'm excited to see this happening. A few things to fix.
@humphd I made some small changes and updated the screenshot in the description |
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.
Awesome.
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.
Well done 👍.
@tpmai22 can you do a rebase instead of merge? |
cc @menghif or @TueeNguyen, if you need help with this rebase. |
@Andrewnt219 @humphd one more reviews and we are good to merged it |
Issue This PR Addresses
Fix #2761
Type of Change
Description
Adding commit messgae in build header for better information about the build
Steps to test the PR
autodeploymentUrl
insrc\api\status\public\js\build-log\check-build-status.js
connectSrc
insrc\api\status\src\server.js
-Change directory tp
api
service and run the front endChecklist