search

ArkProjectNFTs / bridge

The ArkProject Bridge: seamless transfer of NFTs between ETH L1 & Starknet L2. Smart contracts, user-friendly interface, secure & efficient solution. Experience the future of NFT ownership today
https://bridge.arkproject.dev
Apache License 2.0
23 stars 20 forks source link

Fix : #234 Missing checks for address(0x0) when updating address stat… #239

Closed ryzen-xp closed 1 month ago

ryzen-xp commented 1 month ago

…e variables

Summary of Changes Made Input Validation in setL1L2CollectionMapping: Added checks to ensure collectionL1 is not address(0) and collectionL2 is not 0 to prevent invalid mappings. Event Emission: Emitted L1L2CollectionMappingUpdated after setting the mapping for transparency.

vercel[bot] commented 1 month ago

@ryzen-xp is attempting to deploy a commit to the Screenshot Team on Vercel.

A member of the Team first needs to authorize it.

ptisserand commented 1 month ago

Thanks for your PR, could you please add unit test ?

ryzen-xp commented 1 month ago

Ok Sir

ryzen-xp commented 1 month ago

@ptisserand Sir check my PR