-
Notifications
You must be signed in to change notification settings - Fork 2
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
Sovereign-Forge SC #185
Sovereign-Forge SC #185
Conversation
Signed-off-by: Andrei Baltariu <[email protected]>
Signed-off-by: Andrei Baltariu <[email protected]>
Signed-off-by: Andrei Baltariu <[email protected]>
Signed-off-by: Andrei Baltariu <[email protected]>
Signed-off-by: Andrei Baltariu <[email protected]>
Signed-off-by: Andrei Baltariu <[email protected]>
Signed-off-by: Andrei Baltariu <[email protected]>
Signed-off-by: Andrei Baltariu <[email protected]>
Signed-off-by: Andrei Baltariu <[email protected]>
Signed-off-by: Andrei Baltariu <[email protected]>
Signed-off-by: Andrei Baltariu <[email protected]>
Signed-off-by: Andrei Baltariu <[email protected]>
Signed-off-by: Andrei Baltariu <[email protected]>
Signed-off-by: Andrei Baltariu <[email protected]>
Signed-off-by: Andrei Baltariu <[email protected]>
Signed-off-by: Andrei Baltariu <[email protected]>
Signed-off-by: Andrei Baltariu <[email protected]>
Signed-off-by: Andrei Baltariu <[email protected]>
Signed-off-by: Andrei Baltariu <[email protected]>
Signed-off-by: Andrei Baltariu <[email protected]>
Signed-off-by: Andrei Baltariu <[email protected]>
Signed-off-by: Andrei Baltariu <[email protected]>
Coverage SummaryTotals
FilesExpand
|
Contract comparison - from cff6c8a to 60ba9f8
|
fn sovereigns_mapper( | ||
&self, | ||
sovereign_creator: &ManagedAddress, | ||
) -> SingleValueMapper<ChainContractsMap<Self::Api>>; |
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'm thinking we will need to change this mapper later, but I'm not sure.
- maybe we want to display all sovereign chains somewhere without knowing the owners
- also, we will need a way to access all esdt-safe addresses. These are required for explorer to display "cross-chain tx" icon correctly for transactions
For the moment we can keep it SingleValueMapper
.
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.
TODO for different PR
- define a "complete setup phase" for this contract where we check token-handlers and chain-factory storages are set
- only if the setup is completed,
deploy_phase_one
can be called
Signed-off-by: Andrei Baltariu <[email protected]>
Signed-off-by: Andrei Baltariu <[email protected]>
Signed-off-by: Andrei Baltariu <[email protected]>
Co-authored-by: Octavian Axente <[email protected]>
Signed-off-by: Andrei Baltariu <[email protected]>
Fw-upgrade to 0.54.3
Signed-off-by: Andrei Baltariu <[email protected]>
Created the Sovereign-Forge SC that will call the Chain-Factory SC. Basically this contract will be the user's interaction with the Sovereign Chain in order to deploy and setup the needed modules and SCs. This PR contains the following:
phases