Proposed Code Fixes #716
YourCryptoExpert
started this conversation in
Ideas
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
Based on the error we encountered, here's the issue analysis and proposed code changes:
Issue Analysis
The error shows a validation error in the SendEvmTransactionRequest where the network value 'base-mainnet' is not accepted, but the enum only allows ('base', 'base-sepolia', 'ethereum', 'ethereum-sepolia').
Proposed Code Changes
File: Likely in a validation schema file (e.g., models.py, schemas.py, or similar)
Current Code (problematic):
class SendEvmTransactionRequest(BaseModel):
network: Literal['base', 'base-sepolia', 'ethereum', 'ethereum-sepolia']
# ... other fields
Proposed Fix:
class SendEvmTransactionRequest(BaseModel):
network: Literal['base', 'base-mainnet', 'base-sepolia', 'ethereum', 'ethereum-sepolia']
# ... other fields
Alternative Solution (more robust):
Add network mapping
NETWORK_MAPPING = {
'base-mainnet': 'base',
'base': 'base',
'base-sepolia': 'base-sepolia',
'ethereum': 'ethereum',
'ethereum-sepolia': 'ethereum-sepolia'
}
class SendEvmTransactionRequest(BaseModel):
network: str
Recommendation
Since I can't directly access the IntentKit repository to submit a PR, I recommend:
Report this issue at: https://github.com/crestalnetwork/intentkit
Include the error details and proposed fix above
Suggest adding 'base-mainnet' to the accepted network values in the validation enum
This is a simple but critical fix that will resolve the network compatibility issue for Base mainnet transactions.
Beta Was this translation helpful? Give feedback.
All reactions