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

feat: rework documentation #571

Merged
merged 16 commits into from
Jan 26, 2023
Merged

feat: rework documentation #571

merged 16 commits into from
Jan 26, 2023

Conversation

DylanVerstraete
Copy link
Contributor

No description provided.

Copy link
Contributor

@brandonpille brandonpille left a comment

Choose a reason for hiding this comment

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

Looks great. Just one comment.

@@ -1,21 +0,0 @@
# Tfchain terminolog
Copy link
Contributor

@robvanmieghem robvanmieghem Jan 20, 2023

Choose a reason for hiding this comment

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

I know most of this is deprecated but the definition of a Validator or Council member are nowhere else

Copy link
Contributor Author

Choose a reason for hiding this comment

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

What do you mean by this?

@@ -0,0 +1,11 @@
## Production
Copy link
Contributor

Choose a reason for hiding this comment

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

can we also link to ours, also we should use the production release binaries in the repository, also where is our custom spec file?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

why would we need to link to our chainspecs?


## Get the code

```

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's unclear what is commented here..

Choose a reason for hiding this comment

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

For every highlighted command it should be a highlighted bash command like the following:
Screenshot from 2023-01-25 13-36-41


## Installing and running tfchain in dev mode

```

Choose a reason for hiding this comment

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


To wipe data run:

```

Choose a reason for hiding this comment

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


## Compile release build

```

Choose a reason for hiding this comment

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


In a terminal window execute the following command:

```

Choose a reason for hiding this comment

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


In a second terminal window executing the following command:

```

Choose a reason for hiding this comment

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

- Mainnet: 1.0.0 (git checkout 1.0.0)
- Testnet: 1.0.0-b1 (git checkout 1.0.0-b1)
- Devnet: 1.0.0-b9 (git checkout 1.0.0-b9)
Check release page on github to check which binary to use.

Choose a reason for hiding this comment

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

Please link the release page here.


Keys are generated using [subkey](https://substrate.dev/docs/en/knowledgebase/integrate/subkey) or use `tfchain key`.

For each validator generate a key:

Choose a reason for hiding this comment

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

Might be helpful to users to mention what's a validator.

@DylanVerstraete DylanVerstraete merged commit 2725a35 into development Jan 26, 2023
@DylanVerstraete DylanVerstraete deleted the feat_docs branch January 26, 2023 10:46
DylanVerstraete added a commit that referenced this pull request Apr 12, 2023
* feat: rework documentation (#571)

* doc: service consumer contract flow (#567)

* fix: farming policies map ids (#591)

* chore: reset runtime migrations

* feat: delete twin is not allowed (#586)

* fix: typos (#595)

* fix: locked balances (#590)

* feat: allow twin to connect to a relay (#570)

* chore: bump to version 2.3.0-rc1 (specVersion: 124)

* feat: add node power (#592)

* chore: update ADR docs

* chore: bump to version 2.3.0-rc2 (specVersion: 125)

* chore: rework logging pallet smart contract

* fix: allow numbers in relay addresses

* chore: bump to version 2.3.0-rc3 (specVersion: 125)

* fix: migration V15 for devnet #598

* chore: bump to version 2.3.0-rc4 (specVersion: 127)

* chore: restore V15 migration

* chore: update dev chain spec

* fix: soften serial number validation (#603)

* chore: set max length of serial number to 128 bytes

* chore: align tft bridge dependency

* chore: update default dev chainspec

* chore: revert default dev bridge validators chainspec

* Drop unused structopt dependency

This also removes some unused child dependencies

Signed-off-by: Lee Smet <lee.smet@hotmail.com>

* Set the substrate node default chart image to ghcr.io

* fix: rework balances migrations

* chore: refactor

* feat: also remove contract and lock if twin is deleted

* chore: add writes

* chore: resolve comment

* chore: fix ci

* chore: resolve pr comments

* chore: add log

* Update build_test.yaml to use the new runner (#617)

* chore: remove serial valdiation (#616)

* chore: bump to version 2.3.0-rc6 (specVersion: 129)

* chore: rework chart #620

* fix: discount calculation handle variable billing frequency (#626)

* fix: migrations for v2.3.0 (#627)

* chore: bump to version 2.3.0-rc7 (specVersion: 130)

* fix: soften node city/country validation

* chore: bump to version 2.3.0-rc8 (specVersion: 131)

* chore: update cargo lock

* chore: bump version to 2.3.0

---------

Signed-off-by: Lee Smet <lee.smet@hotmail.com>
Co-authored-by: Erwan Renaut <73958772+renauter@users.noreply.github.com>
Co-authored-by: omahs <73983677+omahs@users.noreply.github.com>
Co-authored-by: Lee Smet <lee.smet@hotmail.com>
Co-authored-by: Rob Van Mieghem <robvanmieghem@gmail.com>
Co-authored-by: Brandon <brandon@threefold.tech>
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.

5 participants