OpenZeppelin / cairo-contracts

OpenZeppelin Contracts written in Cairo for Starknet, a decentralized ZK Rollup
https://docs.openzeppelin.com/contracts-cairo
MIT License
813 stars 329 forks source link

Support higher tx versions in AccountComponent #830

Closed martriay closed 8 months ago

martriay commented 10 months ago

Tx version is currently hardcoded in our account implementation, when it should be future proof and accept higher versions. This was originally supported but it was lost in the Cairo 1+ migration due to lack of < and > operators for felt252. We can fix this by casting the tx version into integer types and then comparing.