Skip to content
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.

Sign up for GitHub

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

Jump to bottom

CSM alerts #621

Open
wants to merge 43 commits into
base: main
Choose a base branch
from
Open

CSM alerts #621

wants to merge 43 commits into from

Conversation

madlabman
Copy link
Contributor

@madlabman madlabman commented Oct 9, 2024 •
edited
Loading

According to the list of alerts.

@madlabman madlabman force-pushed the csm-alerts-wip branch 2 times, most recently from 71fee70 to eecbd92 Compare October 10, 2024 14:59
@madlabman madlabman marked this pull request as ready for review October 21, 2024 11:30
@madlabman madlabman requested a review from a team as a code owner October 21, 2024 11:30
vgorkavenko
vgorkavenko previously approved these changes Oct 21, 2024
View reviewed changes
dgusakov
dgusakov requested changes Oct 23, 2024
View reviewed changes
Copy link
Contributor

@dgusakov dgusakov left a comment

Choose a reason for hiding this comment

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

Pls check readme for consistency with the code + check alert emoji so that they match severities

csm-alerts/README.md Outdated Show resolved Hide resolved
csm-alerts/README.md Outdated Show resolved Hide resolved
csm-alerts/README.md Outdated Show resolved Hide resolved
csm-alerts/README.md Outdated Show resolved Hide resolved
csm-alerts/README.md Outdated Show resolved Hide resolved
csm-alerts/src/services/EventsWatcher/events/module.ts Outdated Show resolved Hide resolved
csm-alerts/src/services/EventsWatcher/events/module.ts Outdated Show resolved Hide resolved
csm-alerts/src/services/EventsWatcher/events/module.ts Outdated Show resolved Hide resolved
csm-alerts/src/services/EventsWatcher/events/module.ts Outdated Show resolved Hide resolved
csm-alerts/src/services/EventsWatcher/events/oracle.ts Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@skhomuti skhomuti Awaiting requested review from skhomuti skhomuti is a code owner automatically assigned from lidofinance/community-staking

@dgusakov dgusakov Awaiting requested review from dgusakov

@vgorkavenko vgorkavenko Awaiting requested review from vgorkavenko

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@madlabman @vgorkavenko @dgusakov @larmork