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 #2415: add live build logs in status dashboard #2455
Fix #2415: add live build logs in status dashboard #2455
Conversation
19e812d
to
45e40cb
Compare
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.
We need to fix #2454 for the dev setup.
Also, let's get a link from the main dashboard index.html
to build.html
.
45e40cb
to
c660380
Compare
c660380
to
3cbb451
Compare
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.
Looking close to me
@Andrewnt219 can you squash/rebase this for me? I think it's basically ready. I just want one more review on it. cc @manekenpix, in case you want to read it before it goes in. |
port build log from React version move scripts down remove protocol refactor autodeploymentUrl remove class from body
8f58dd2
to
29d8141
Compare
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.
Can't wait for this to finally happen! I've been wanting to do it for over a year, and never had time to finish it. Thanks for taking it on.
This's just the first step. Let's deploy it to see if it's everything's working and I'll continue to iterate over this feature until it's finished 👍. |
Issue This PR Addresses
Closes #2415
Type of Change
Description
On /build.html, periodically fetches build status and shows build logs (if any) using xterm.
Test plan
To test locally, change to
<base href="/" />
inside build.html and run start script. The site is onlocalhost:1111/pages/build.html
.Xterm UI should show up. In the dev console,
/status
is requested periodically.Checklist