Skip to content
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

Add a TransferOwnership use case #24

Open
luislucena16 opened this issue Feb 15, 2024 · 7 comments
Open

Add a TransferOwnership use case #24

luislucena16 opened this issue Feb 15, 2024 · 7 comments

Comments

@luislucena16
Copy link

hey @d-xo I think it would be great to add an example related to ownership to validate that the direction the new control is taking is not a contract, what do you think? :)

@SIDHARTH20K4
Copy link

@luislucena16 can i work on this issue?!

@luislucena16
Copy link
Author

luislucena16 commented Feb 22, 2024

hey @SIDHARTH20K4 yes, no problem, you can do it!
link the PR to this issue so we don't create another one! so we can review it :)

@SIDHARTH20K4
Copy link

hey @SIDHARTH20K4 yes, no problem, you can do it! link the PR to this issue so we don't create another one! so we can review it :)

could you please elaborate the problem... so that I can I can work on it freely

@luislucena16
Copy link
Author

luislucena16 commented Feb 25, 2024

@SIDHARTH20K4 I think you can make a simple deposit and withdraw contract that includes the transferOwnership and how this would be protected

@SIDHARTH20K4
Copy link

I've created a PR
kindly check it and let me know if everything is perfect!
@d-xo

@luislucena16
Copy link
Author

I left you some comments in the PR!

@SIDHARTH20K4
Copy link

I left you some comments in the PR!

I've updated it!! Kindly check and let me know if averything is perfect!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants