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

helper contracts for separating logic and data #9

Closed
maraoz opened this issue Aug 19, 2016 · 2 comments
Closed

helper contracts for separating logic and data #9

maraoz opened this issue Aug 19, 2016 · 2 comments
Labels
feature New contracts, functions, or helpers.

Comments

@maraoz
Copy link
Contributor

maraoz commented Aug 19, 2016

Idea from https://youtu.be/1D7BH8krdhQ?t=11m54s

Some of this is already present in Ownable.sol, but needs more thought

@maraoz maraoz added feature New contracts, functions, or helpers. research labels Dec 19, 2016
@come-maiz
Copy link
Contributor

I think people can apply this pattern with what we have in the repo. He mentions transfer ownership, that's Ownable. He mentions transferring stuff before self destruct, https://github.com/OpenZeppelin/openzeppelin-solidity/blob/v1.9.0/contracts/lifecycle/TokenDestructible.sol shows how to do that. With these, somebody can make a contract for data and another that allows to transfer the ownership of that data.

He also mentions a generic storage. We have been experimenting with that in the labs on EternalStorage, but I'm not sure if we want to move something like that into openzeppelin.

I also don't think it would be a good idea to apply this pattern to all the contracts in openzeppelin; and most of his concerns will be solved by the upgradeability in zeppelinOS.

I'm not going to close this issue (yet), in case I'm missing some details that we would like to have in the framework.

@frangio frangio removed the backlog label Jun 15, 2018
@nventuro
Copy link
Contributor

nventuro commented Mar 7, 2019

I think proxy contracts (such as ZeppelinOS's) are a more robust solution to this issue, and should advocate for those when this requirement is present.

@nventuro nventuro closed this as completed Mar 7, 2019
djvex pushed a commit to djvex/openzeppelin-contracts that referenced this issue Jan 1, 2022
Add RPC: eth_blockNumber, eth_getBalance, eth_getBlockByHash, eth_get…
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New contracts, functions, or helpers.
Projects
None yet
Development

No branches or pull requests

5 participants