Missing Public Burn Function in erc721_bridgeable.cairo Limits Future Compatibility

[ptisserand]

From https://codehawks.cyfrin.io/c/2024-07-ark-project/s/729

The erc721_bridgeable.cairo contract lacks a public burn function, which is essential for enabling token burning on Layer 2 (L2) in future updates, particularly with the planned useAutoBurn strategy.

[saimeunt]

@ptisserand Would love to work on this one, I have some experience in Cairo including ERC721Burnable logic 👌
https://github.com/carbonable-labs/cairo-erc-7498/blob/main/src/erc7498/interface.cairo#L47-L49

[mubarak23]

I am applying to this issue via OnlyDust platform.

My background and how it can be leveraged

I am a Experience Cairo smart contract developer with experience working on projects such as Just Art Peace, Dojo, Kart, TBA, and Shinigami. Before transitioning to Cairo development, I was a backend developer specializing in Rust.

My recent work with cairo starknet

• check for coinbase maturity keep-starknet-strange/raito#128
• validate relative lock time on input of transaction keep-starknet-strange/raito#143
• lock and is_lock function of lockable component horuslabsio/TBA#45
• feat: _asReferencePubParams check horuslabsio/coloniz-core#60
• Ft nft mint quest keep-starknet-strange/art-peace#121
• feat: nullifier_registry contract functions keep-starknet-strange/zkramp#33
• Ft username store contract keep-starknet-strange/art-peace#32
• Account v3 test coverage horuslabsio/TBA#53

My recent work with rust

• https://github.com/mubarak23/axum-user-auth

• remove debug info from emitting antoyo/rustc_codegen_gcc#1

• Remove pass test antoyo/rustc_codegen_gcc#5

• https://github.com/mubarak23/soroban-bank

How I plan on tackling this issue

AS the issue describe, am going to add a function that enable enabling token burning on Layer 2 and also write test case for the function sure it’s work as expected.

ETH: 16HRS

[raizo07]

I am applying to this issue via OnlyDust platform.

My background and how it can be leveraged

Hello I'll like to be assigned to work on this.

I'm a software Dev with over four years experience and have worked on a couple of projects here. Here's a link to my profile https://app.onlydust.com/u/raizo07

[aji70]

I am applying to this issue via OnlyDust platform.

My background and how it can be leveraged

i'm a solidity and cairo smart contract developer with over 2 years experience and believe i have the skill set for the task

How I plan on tackling this issue

i would create and add the burn function needed making it public to enhance the performance of the contract

[saimeunt]

I am applying to this issue via OnlyDust platform.

My background and how it can be leveraged

Would love to work on this one, I have some experience in Cairo including ERC721Burnable logic 👌
https://github.com/carbonable-labs/cairo-erc-7498/blob/main/src/erc7498/interface.cairo#L47-L49

How I plan on tackling this issue

I will implement a burn function on ERC721Bridgeable and provide all the necessary tests.

[onlydustapp]

The maintainer ptisserand has assigned mubarak23 to this issue via OnlyDust Platform.
Good luck!

[ptisserand]

Closed by #243