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

Added NativeBalances.sol to ensure trackablility of Native balances #386

Merged
merged 4 commits into from
Feb 10, 2025

Conversation

mrk-hub
Copy link
Contributor

@mrk-hub mrk-hub commented Feb 10, 2025

New Smart Contract,
NativeBalances.sol
-view only smart contract to track native balances of passed in wallets.

Copy link

octane-security-app bot commented Feb 10, 2025

Summary by Octane

New Contracts

  • NativeBalances.sol: The smart contract retrieves and returns native cryptocurrency balances for a single address or multiple addresses.

Updated Contracts

No contracts were updated in this PR.


🔗 Commit Hash: 148f6bf

Copy link

octane-security-app bot commented Feb 10, 2025

Overview

Octane AI analysis has finished. No vulnerabilities were found. Cheers! 🎉🎉🎉


🔗 Commit Hash: 148f6bf

karacurt
karacurt previously approved these changes Feb 10, 2025
Copy link
Contributor

@karacurt karacurt left a comment

Choose a reason for hiding this comment

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

lg

found 2 typos

web3/contracts/utils/NativeBalances.sol Outdated Show resolved Hide resolved
web3/scripts/deploy-native-balances.md Outdated Show resolved Hide resolved
Copy link
Contributor

@karacurt karacurt left a comment

Choose a reason for hiding this comment

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

🚀

@mrk-hub mrk-hub merged commit af7d5ed into main Feb 10, 2025
3 checks passed
@mrk-hub mrk-hub deleted the native-balances-contract branch February 10, 2025 21:26
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.

2 participants