-
Notifications
You must be signed in to change notification settings - Fork 264
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
add platform # link #1678
base: master
Are you sure you want to change the base?
add platform # link #1678
Conversation
3839b1a
to
a187ab9
Compare
a187ab9
to
5c6c46e
Compare
e244a24
to
6b20084
Compare
6b20084
to
51a469b
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.
Requested a small change in phrasing + had a comment on the video. Can you fix the first and then merge? No need to change the video before merging.
[Installation instructions »](https://docs.robusta.dev/master/setup-robusta/installation/index.html) | ||
|
||
<!-- <p align="right">(<a href="#top">back to top</a>)</p> --> | ||
|
||
## 🖥 Free Robusta UI | ||
Take your Kubernetes monitoring to the next level with the [Robusta SaaS platform](https://platform.robusta.dev/#?utm_source=github&utm_medium=robusta-readme&utm_content=free_robusta_ui_section). Creating an account is free, and you can: |
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 think it should be "Creating an account is free, and includes:" as below each bullet lists a feature not something the user can do (I know you include it in parenthesis but the list is a list of features)
- **Alert Timeline**: View Prometheus alerts across multiple clusters and spot correlations with a powerful timeline view | ||
- **Change Tracking**: Correlate alerts with changes to your infrastructure or applications, with Robusta’s automatic change tracking for Kubernetes | ||
|
||
<a href="https://www.loom.com/share/9c40ab750e5c4dfabd80f0f6b65f3813"> |
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.
It would be better to highlight in the video why each feature is useful, not just what the feature is. E.g. instead of 'Change Tracking' show 'Correlate alerts with changes to your environment'. (On the AI Assistant I think its pretty much fine as its obvious.)
That said, no need to hold up the PR for this. Please merge and if you have a chance fix it afterwards.
No description provided.