-
Notifications
You must be signed in to change notification settings - Fork 247
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
Kernel: allow initializing app proxies on overloaded newAppInstance #415
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
❤️
contracts/kernel/Kernel.sol
Outdated
@@ -70,15 +70,16 @@ contract Kernel is IKernel, KernelStorage, Petrifiable, IsContract, VaultRecover | |||
* @param _setDefault Whether the app proxy app is the default one. | |||
* Useful when the Kernel needs to know of an instance of a particular app, | |||
* like Vault for escape hatch mechanism. | |||
* @param _initializePayload Payload for call made by the proxy on its constructor to initialize |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thoughts on this going before setDefault
, since it's part of the proxy initialization?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have no hard feelings either way.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done in f4bb44c
AppProxyFactory
exposes the ability to pass an initialization payload that app proxies execute on their constructor. This PR exposes this ability at the Kernel level, so apps can be atomically initialized when they are created.