Skip to content

feat(site): add health warning and a health monitor page #8844

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

Merged
merged 20 commits into from
Aug 2, 2023

Conversation

BrunoQuaresma
Copy link
Collaborator

@BrunoQuaresma BrunoQuaresma commented Aug 1, 2023

Screen Shot 2023-08-01 at 13 34 28 Screen Shot 2023-08-01 at 13 34 40

@BrunoQuaresma BrunoQuaresma self-assigned this Aug 1, 2023
@bpmct
Copy link
Member

bpmct commented Aug 1, 2023

Close #7748

Let's leave the issue open until it's removed from experimental.

@bpmct
Copy link
Member

bpmct commented Aug 1, 2023

/deploy-pr

@github-actions
Copy link

github-actions bot commented Aug 1, 2023

🚀 Deploying PR 8844 ...
⚠️ This deployment will be deleted when the PR is closed.

@BrunoQuaresma
Copy link
Collaborator Author

@bpmct ok! About the deploy-pr Idk how to activate an experimental flag to it.

@coder coder deleted a comment from github-actions bot Aug 2, 2023
@coder coder deleted a comment from github-actions bot Aug 2, 2023
@BrunoQuaresma BrunoQuaresma requested a review from mtojek August 2, 2023 13:54
Copy link
Member

@mtojek mtojek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like linter is also complaining about:

2277 |   coder_version: "v0.27.1-devel+c575292ba",

@BrunoQuaresma BrunoQuaresma requested review from mtojek and matifali and removed request for kylecarbs August 2, 2023 16:19
@bpmct
Copy link
Member

bpmct commented Aug 2, 2023

/deploy-pr

@github-actions
Copy link

github-actions bot commented Aug 2, 2023

🚀 Deploying PR 8844 ...
⚠️ This deployment will be deleted when the PR is closed.

@matifali
Copy link
Member

matifali commented Aug 2, 2023

@bpmct you also need to specify experiments

Copy link
Member

@bpmct bpmct left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's bring this one in :) It's under experimental

@BrunoQuaresma BrunoQuaresma merged commit cf35c0d into main Aug 2, 2023
@BrunoQuaresma BrunoQuaresma deleted the bq/health-page branch August 2, 2023 17:49
@github-actions github-actions bot locked and limited conversation to collaborators Aug 2, 2023
@coder coder deleted a comment from github-actions bot Aug 4, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants