-
Notifications
You must be signed in to change notification settings - Fork 46
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
[feat] Implement the functionality of acceptProfileOwnership #8
Comments
I would like to take on this |
Hello! I would like to help with this one! It can take me about a day to complete, I'm doing my first contributions and I want to improve my Cairo skills, completing this issue can help me! |
hey there @Akashneelesh, I would like to take this one up. |
Hello my name is Kenneth Komu. I am a Starknet developer and I would like to work on the issue. |
Would love to work on this |
Heyy @Thirumurugan7 , you have been assigned this issue less goo !! @danielcdz @Sh0g0-1758 @Josh-121 and @kenkomu since we work on fcfs basis, I have assigned this issue to @Thirumurugan7. There are alot more issues coming up ! |
Down below is the function that is to be implemented in the contract but in cairo.
https://github.com/allo-protocol/allo-v2/blob/4dd0ea34a504a16ac90e80f49a5570b8be9b30e9/contracts/core/Registry.sol#L267
Inorder to do something similar to delete functionality on Starknet -> You will have to just change the values to 0.
Resources :
https://book.cairo-lang.org/
https://book.cairo-lang.org/ch08-01-generic-data-types.html?highlight=struct#structs
Unruggable memecoin : https://github.com/keep-starknet-strange/unruggable.meme/tree/main/packages/contracts/src
Blobstream : https://github.com/keep-starknet-strange/blobstream-starknet/tree/main/src
Exclusively for Starknet India team - To onboard new devs from India
Estimated Time taken to complete - 2-3 days (Max)
The text was updated successfully, but these errors were encountered: