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.

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

Implement Tenderly Web3 Actions to monitor different RedStone smart contracts #212

Merged

Conversation

matjazv
Copy link
Contributor

@matjazv matjazv commented Nov 15, 2024

What was the problem?

This PR resolves #LISK-1157.

How was it solved?

  • Web3 Actions for USDT/USD, LSK/USD and ETH/USD token pairs were created to monitor their update
  • It's possible to deploy those actions using Tenderly CLI

How was it tested?

Web3 Actions for all three token pairs were deployed to Tenderly and they were manually executed to check if alerts inside Opsgenie were created.

@matjazv matjazv self-assigned this Nov 15, 2024
@matjazv matjazv changed the title Lisk 1157 set up alerts on tenderly and or elastic Implement Tenderly Web3 Actions to monitor different RedStone smart contracts Nov 15, 2024
@matjazv matjazv force-pushed the LISK-1157-Set-up-alerts-on-Tenderly-and-or-Elastic branch from 7d5eaa5 to 148ecd0 Compare November 15, 2024 16:21
web3-actions/priceFeedMonitor/README.md Outdated Show resolved Hide resolved
web3-actions/priceFeedMonitor/README.md Outdated Show resolved Hide resolved
web3-actions/priceFeedMonitor/README.md Outdated Show resolved Hide resolved
web3-actions/priceFeedMonitor/README.md Outdated Show resolved Hide resolved
web3-actions/priceFeedMonitor/ethUsd.yaml Outdated Show resolved Hide resolved
@matjazv matjazv force-pushed the LISK-1157-Set-up-alerts-on-Tenderly-and-or-Elastic branch from 4d01af8 to f1ce796 Compare November 18, 2024 14:58
@matjazv
Copy link
Contributor Author

matjazv commented Nov 18, 2024

@sameersubudhi
All your comments were resolved inside commit f1ce796.

Copy link
Member

@sameersubudhi sameersubudhi left a comment

Choose a reason for hiding this comment

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

Looks good overall. However, please consider the following minor observations.

web3-actions/priceFeedMonitor/README.md Outdated Show resolved Hide resolved
web3-actions/priceFeedMonitor/tokenPairMonitor/ethUsd.ts Outdated Show resolved Hide resolved
web3-actions/priceFeedMonitor/tokenPairMonitor/lskUsd.ts Outdated Show resolved Hide resolved
web3-actions/priceFeedMonitor/tokenPairMonitor/usdtUsd.ts Outdated Show resolved Hide resolved
web3-actions/priceFeedMonitor/ethUsd.yaml Outdated Show resolved Hide resolved
web3-actions/priceFeedMonitor/lskUsd.yaml Outdated Show resolved Hide resolved
web3-actions/priceFeedMonitor/usdtUsd.yaml Outdated Show resolved Hide resolved
@sameersubudhi sameersubudhi merged commit fef905d into development Nov 18, 2024
4 checks passed
@sameersubudhi sameersubudhi deleted the LISK-1157-Set-up-alerts-on-Tenderly-and-or-Elastic branch November 18, 2024 21:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants