-
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
merge fix/paymaster into develop #78
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Collaborator
dappsar
commented
Feb 21, 2025
•
edited
Loading
edited
- [fix] improved paymaster verification, and security
- [feat] added paymaster test, improved custom errors in paymaster cont…
- [chore] added extra logs to paymaster deploy
- [feat] added paymaster only deploy to makefile
- [fix] removed fee update, added initialization params
- [fix] compiler warnings / errors
- [fix] compiler errors
- [fix] compiler warnings
- [feat] updated tests and removed admin fee rule
- [wip] tests update
- [fix] correct proxy initialization parameters in ChatterPayWalletFactory
- [fix] enhance proxy safety and add structured storage pattern
- [fix] adapt deploy factory to the new implementation
- [fix] adapt deploy factory (individual script) to the new implementation
- [fix] remove redundant token whitelisting step
- [feat] added getters
- [docs] 📝 update uniswap addresses in example_env
- [feat] ✨ add logic to choose whether to deploy paymaster or not
- [chore] 🔊 update logs/comments in DeployAllContracts
- [feat] ✨ add paymaster address env var in example_env
- [test] 🔨 add script to run only one test by name
- [test] 🔨 add script in package.json to call on-test script
- [chore] 🧑💻 print contracts addresses deployed in bdd format
- [ci] 💩 skip validatePaymasterUserOp tests until fix related errors
- [chore] 🔊 add logs in testOwnerShipManagement test
- [chore] 🧪 replace function with onlyOwner to test in testOwnerShipManagement
- [chore] 🧪 update numbre to create temp wallet in testSignatureValidation
- [bug] 💩 skip validatePaymasterUserOp funciton in contract until errors are fixed
- [feat] ✨ add stake functions in Paymaster
- [feat] ✨ cast entrypoint address to iEntryPoint
- [docs] 📝 add Paymaster stake instructions in documentation
- [chore] 🔊 add logs entry as a reminder to stake balance in Paymaster
- [chore] 🔊 add print to console as a reminder to copy EntryPoint ABI
- [feat] ✨ add events in init/end executeTokenTransfer
- [feat] ✨ replace transfer by safeTransfer in executeTokenTransfer
- [feat] ✨ update balance validation in executeTokenTransfer
- [feat] ✨ improve function _payPrefund in Chatterpay contract
- [fix] 🐛 update and split logic in validateUserUp
- [chore] ♻️ disable constructor with _disableInitializers in Chatterpay
- [refactor] ♻️ update variable name newowner to owner
- [chore] 💡 add comment reminder to improve swap token list in Chatterpay.sol
- Add ChatterPayStorage for explicit storage layout - Implement secure storage slot pattern - Add implementation contract initialization protection - Increase storage gap for safer upgrades - Add implementation slot for UUPS pattern
This was
linked to
issues
Feb 21, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
bug
Something isn't working
documentation
Improvements or additions to documentation
enhancement
New feature or request
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.