-
Notifications
You must be signed in to change notification settings - Fork 1
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
Ready: monitor the funding account of vortex #277
Ready: monitor the funding account of vortex #277
Conversation
✅ Deploy Preview for pendulum-pay ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@Sharqiewicz can you please merge or rebase the latest |
…-account-of-vortex
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.
Looks good overall, though I would appreciate if we add some mechanism that limits the number of duplicate messages emitted to one per .. 6 hours maybe? So we avoid spamming the channel. Each type of warning should have its own timer though.
@pendulum-chain/devs Ready for review ✅ |
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.
Looks good ! Only left one small comment.
Did you test it or do we test locally first?
🦈 🟢 PR
@slack/web-api
SLACK_WEB_HOOK_TOKEN
env variablemoonbeam
(GLMR),pendulum
(EUR, USDC.axl, PEN),stellar
(XLM)