You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This should really be part of the first release to avoid all the problems with hard-coding contract addresses everywhere.
However, it does make deploying the contracts a little more difficult because you must associate them with a specific registry, deploy with the same account the registry was deployed with etc. It also introduces problems if you want to deploy multiples of the same contract.
Another option would be a contract factory with auto-registration... but again, adds complexities.
The problem I was having which lead to wanting this Registry contract was that whenever the migrations file changed the contract addresses would also change (which breaks stuff), this is a problem during development, but may also be a problem when deploying to the test networks.
The text was updated successfully, but these errors were encountered:
The Registrar / Registry contract makes it easier to work with related and interlinked contracts.
See: https://github.com/HarryR/panautomata/blob/61a4bdaa26fe3dfb0305606b1d143df6c9180061/solidity/contracts/Registrar.sol
This should really be part of the first release to avoid all the problems with hard-coding contract addresses everywhere.
However, it does make deploying the contracts a little more difficult because you must associate them with a specific registry, deploy with the same account the registry was deployed with etc. It also introduces problems if you want to deploy multiples of the same contract.
Another option would be a contract factory with auto-registration... but again, adds complexities.
The problem I was having which lead to wanting this Registry contract was that whenever the migrations file changed the contract addresses would also change (which breaks stuff), this is a problem during development, but may also be a problem when deploying to the test networks.
The text was updated successfully, but these errors were encountered: