Skip to content
This repository has been archived by the owner on Dec 2, 2024. It is now read-only.

public testnet instructions #83

Merged
merged 4 commits into from
Nov 5, 2021
Merged

public testnet instructions #83

merged 4 commits into from
Nov 5, 2021

Conversation

silky
Copy link
Contributor

@silky silky commented Nov 5, 2021

Fixes SCP-3021

I think the instructions need a note to wait for the chain-index to say it's ready to go; otherwise the entire thing fails, silently.

TODO:

  • Put the private testnet intructions in here as well
  • Verify them again
  • Add a gif showing it in operation
  • Add Jann's notes about why it may go wrong.

Pre-submit checklist:

  • Branch
    • Tests are provided (if possible)
    • Commit sequence broadly makes sense
    • Key commits have useful messages
    • Relevant tickets are mentioned in commit messages
    • Formatting, materialized Nix files, PNG optimization, etc. are updated
  • PR
    • Self-reviewed the diff
    • Useful pull request description
    • Reviewer requested

Comment on lines +58 to +66
- Save this in a file named `restore-wallet.json` in the `testnet` folder:

```
> cd $PLUTUS/plutus-pab/test-node/
> cat testnet/restore-wallet.json
{ "name": "PAB testing wallet"
, "mnemonic_sentence": ["word1", "word2", ...]
, "passphrase": "pab123456789"
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe a better way to put the recovery phrase? I had to manually add the double quotes and commas..

Copy link
Contributor Author

@silky silky Nov 5, 2021

Choose a reason for hiding this comment

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

Yes; I ended up using a search-and-replace: of space ( ) to ", " 😀

Copy link
Contributor

Choose a reason for hiding this comment

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

Oh right, that works too :D

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I did do it manually the first few times 😁

> export WALLET_ID=...
```

- Put some tAda in it, by first obtaining an address:
Copy link
Contributor

Choose a reason for hiding this comment

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

Correction: Ada

Copy link
Contributor

Choose a reason for hiding this comment

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

Ah, unless tAda means testnet Ada

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, I discovered this yesterday!


```
> cd $PLUTUS/plutus-pab/test-node/
> cabal exec -- plutus-chain-index --config testnet/chain-index-config.json start-index
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm getting a testnet/chain-index-config.json not found.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@koslambrou It should be there; did you run that command from the right dir? (note the file is added in this diff)

Copy link
Contributor

Choose a reason for hiding this comment

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

Oops, I was on the wrong branch

@silky silky merged commit 404af7a into main Nov 5, 2021
@silky silky deleted the public-testnet-e2e-docs branch November 5, 2021 14:33
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.

2 participants