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

Core Contract Improvements & Security Enhancements #62

Merged
merged 42 commits into from
Jan 24, 2025

Conversation

TomasDmArg
Copy link
Collaborator

@TomasDmArg TomasDmArg commented Jan 23, 2025

Description

This PR introduces major improvements to the ChatterPay smart contracts:

🏗 Infrastructure

  • Reorganized folder structure
  • Added integration tests
  • Created test automation script
  • Created core contracts ABI generation script
  • Switched from API3 to Chainlink oracles

🔒 Security

  • Limited unsafe execute function for transfers
  • Added fee management system
  • Implemented admin controls
  • Enhanced error handling

⚡️ Features

  • Added Uniswap integration (executeSwap)
  • Implemented batch transfers
  • Added price feed integration
  • Finished fee collection and single transfers
  • Enhanced fee calculations

📚 Documentation

  • Added comprehensive docs for all contracts
  • Documented testing strategy
  • Added setup guides

🚨Breaking Changes

  • Limited public execute function for transfers (modified transfers)
  • Changed oracle integration from API3 to Chainlink
  • Modified swap approach
  • Added extra approach for transferAll
  • Modified fee calculation mechanism

🧪Test Coverage

✅ Unit tests
✅ Integration tests
✅ Gas optimization tests
✅ Security tests

Resolves: #53, #54, #55, #56, #57, #58, #59, #60, #61, #63

TomasDmArg and others added 30 commits January 5, 2025 22:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment