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

Proposal for a new level [33] [BankBuilder]. #722

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

orangesantra
Copy link

  • BankBuilder is the contract used to deploy Bank Contract. When BankBuilder instance is created, Bank contract is also
    deployed with 0.001 ethers.

  • Bank Contract can deploy Recipient contract.

  • Bank Contract has grouped the list of 10 addresses of different Recipient contracts in order, but not deployed yet. The address
    of Recipient contract to be deployed first is at position no. 1 on list, the address to be deployed second is on position no. 2 on
    list and so on.

  • Accidently when Bank Contract is deployed all the funds in Bank Contract is transferred to 5th Recipient address.
    Now the funds are stuck on 5th Recipient address.

The Goal is to retrieve the funds back to BankBuilder contract !

@orangesantra
Copy link
Author

Concerned openzeppelin team, please suggest the improvements in PR , so i can work on them.

@orangesantra orangesantra changed the title Proposal for a new level [30] [BankBuilder]. Proposal for a new level [33] [BankBuilder]. Apr 24, 2024
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

Successfully merging this pull request may close these issues.

1 participant