-
Notifications
You must be signed in to change notification settings - Fork 0
Implementation of JPYC Python SDK #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
Conversation
|
@Mameta29 |
Mameta29
left a comment
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.
セルフレビュー
| Returns: | ||
| HexBytes: トランザクションハッシュ | ||
| """ | ||
| ... |
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.
日本語のコメントが混ざっているので英語で統一する
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.
コメントが日本語になっているので英語 or 削除
| """ | ||
| return await asyncio.to_thread(func) | ||
|
|
||
| async def _execute_transaction(self, contract_fn: ContractFunction) -> HexBytes: |
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.
エラーハンドリングの部分で、具体的なエラータイプを追加したほうがよいか。例えば InsufficientFundsError や GasLimitExceededError など。
| self.web3 = get_web3_for_chain(chain_name, network_name, rpc_endpoint) | ||
| self.account = None | ||
|
|
||
| def create_private_key_account(self, private_key: Optional[str] = None) -> Optional[LocalAccount]: |
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.
こちらは常にLocalAccountを返すので、戻り値の型をOptional[LocalAccount]ではなく単にLocalAccountにする
| @@ -0,0 +1,65 @@ | |||
| [tool.poetry] | |||
| name = "jpyc-sdk-python" | |||
| version = "1.0.0" | |||
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.
このバージョンの書き方はnode sdkにならっている
|
一旦こちらの PR は閉じさせていただきます 🙇🏻 |
🎨 Overview
Implementation of a Python SDK for interacting with JPYC tokens. This SDK provides interfaces to interact with JPYCv2 smart contracts across multiple chains including Ethereum, Polygon, Avalanche, and Astar.
🌈 Details
SDK implementation compatible with Python 3.11 and above
Smart contract integration using web3.py
Support for multiple chains (Ethereum, Polygon, Gnosis, Avalanche, Astar, Shiden, etc.)
Implementation of all API endpoints for JWT authentication and token operations
Unit test implementation and documentation generation
GitHub workflow configuration
📚 References
JPYCv2 Contracts
ERC-20 Specification
RetryClaude does not have internet access. Links provided may not be accurate or up to date.Claude can make mistakes. Please double-check responses.