Skip to content
This repository has been archived by the owner on Jan 22, 2025. It is now read-only.

Simple client demo #470

Merged
merged 5 commits into from
Jun 27, 2018
Merged

Conversation

CriesofCarrots
Copy link
Contributor

Toward #343
The user interaction could be automated, if that type of demo is preferred.

This does not seem to work with testnet.solana.com at present, but does work with a local leader/drone.

@mvines
Copy link
Contributor

mvines commented Jun 27, 2018

Neat, can you please add a simple-client-demo entry to snapcraft.yaml too, like https://github.com/solana-labs/solana/blob/master/snap/snapcraft.yaml#L48

@@ -0,0 +1,21 @@
#!/bin/bash
#
# usage: $0 <rsync network path to solana repo on leader machine> <number of nodes in the network???>"
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove “number of nodes in the network” argument documentation?

@garious
Copy link
Contributor

garious commented Jun 27, 2018

Let's dumb it down. No Rayon, no GenKeys, pay provided amount to provided pubkey from provided private key. Formats for keys need to be discussed, but just serde JSON will be fine to get this PR merged. Feel free to pipe in a renamed mint.json (id.json?) for that private key. See solana-genesis for that code.

@garious
Copy link
Contributor

garious commented Jun 27, 2018

Paying should also print the signature. Next step will be to check if that signature is in the bank.

@garious
Copy link
Contributor

garious commented Jun 27, 2018

And can you rename to wallet-demo?

@garious
Copy link
Contributor

garious commented Jun 27, 2018

Everything I requested can come in as a separate PR. Merging to ensure you don't end up in rebase hell.

@garious garious merged commit 6dac87f into solana-labs:master Jun 27, 2018
@CriesofCarrots
Copy link
Contributor Author

Thanks, @garious !

@CriesofCarrots CriesofCarrots deleted the simple_client_demo branch June 28, 2018 15:10
lijunwangs pushed a commit to lijunwangs/solana that referenced this pull request Apr 4, 2024
* ff cleanup: reduce_stake_warmup_cooldown

* update instruction comments to indicate stake config is unused
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants