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

pre-genesis wallet #870

Closed
wants to merge 9 commits into from
Closed

pre-genesis wallet #870

wants to merge 9 commits into from

Conversation

tzemanovic
Copy link
Member

@tzemanovic tzemanovic commented Dec 7, 2022

based on #849, this PR adds --pre-genesis argument to the wallet commands to allow to generate keys, implicit addresses and MASP keys without having a chain setup - this is a needed for #685. If no chain is setup yet (i.e. there's no .namada or it's empty), the wallet defaults to use the pre-genesis wallet even without the --pre-genesis flag. The pre-genesis wallet is located in .namada/pre-genesis/wallet.toml.

relevant commits: https://github.com/anoma/namada/pull/870/files/0f7da43aa22d12d86e917a07f80f0248a15df711..0b3b35706768a98dcc1eb844cf5d706d89f41905

closes #784

@@ -165,22 +216,9 @@ impl Context {

/// Get the wasm directory configured for the chain.
///
/// Note that in "dev" build, this may be the root `wasm` dir.
/// TODO(Tomas): Note that in "dev" build, this may be the root `wasm` dir.
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is left for #685 where we're removing the "dev" build

tzemanovic added a commit that referenced this pull request Dec 7, 2022
@tzemanovic tzemanovic marked this pull request as ready for review December 8, 2022 09:28
tzemanovic added a commit that referenced this pull request Dec 8, 2022
@juped juped added this to the 0.14 milestone Jan 10, 2023
tzemanovic added a commit that referenced this pull request Jan 12, 2023
@tzemanovic tzemanovic modified the milestones: 0.14, 0.15 Feb 8, 2023
@juped juped removed this from the 0.15 milestone May 15, 2023
@cwgoes cwgoes mentioned this pull request May 18, 2023
10 tasks
@tzemanovic tzemanovic marked this pull request as draft May 26, 2023 14:41
@tzemanovic
Copy link
Member Author

@Fraccaman this depends on #849 which has breaking changes for genesis, so let's leave it until #685 is also ready. This PR will need to be updated for SDK

@tzemanovic
Copy link
Member Author

replaced by #1782

@tzemanovic tzemanovic closed this Aug 3, 2023
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.

interacting with the cli requires a node locally
2 participants