-
Notifications
You must be signed in to change notification settings - Fork 28
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
revert from using abigen! macro #599
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
WalkthroughThe recent changes introduce a comprehensive implementation of the ERC-20 token standard for the StarkNet blockchain, encapsulated in a new Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant Erc20
participant StarkNet
User->>Erc20: Create new token instance
Erc20->>StarkNet: Set contract address
User->>Erc20: Call balanceOf
Erc20->>StarkNet: Query balance
StarkNet-->>Erc20: Return balance
Erc20-->>User: Provide balance info
User->>Erc20: Transfer tokens
Erc20->>StarkNet: Execute transfer
StarkNet-->>Erc20: Confirm transfer
Erc20-->>User: Transfer successful
TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configuration File (
|
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.
Actionable comments posted: 1
basically a revert of #562 because of an issue during the
abigen!
proc macro expansion, where downstream crates arent able to resolve the path correctly because the path ingets resolved relative to the downstream crate during compilation which results in file not found error.
another approach to allow using
abigen!
macro is to use the#![feature(proc_macro_expand)]
feature which requires nightly rust.Summary by CodeRabbit
New Features
Erc20Reader
for read-only access to token states, enhancing user experience with efficient querying capabilities.Bug Fixes
Refactor
controller
anderc_20
, indicating a shift in how contract bindings will be managed in the future.