Skip to content

Post genesis claim#431

Merged
lsaether merged 39 commits intomasterfrom
post-genesis-claim
May 26, 2020
Merged

Post genesis claim#431
lsaether merged 39 commits intomasterfrom
post-genesis-claim

Conversation

@ansonla3
Copy link
Copy Markdown
Contributor

@ansonla3 ansonla3 commented May 5, 2020

closes #396
closes #452

Images still need to be updated.

@ansonla3 ansonla3 added the WIP label May 5, 2020
@ansonla3 ansonla3 linked an issue May 5, 2020 that may be closed by this pull request
Comment thread docs/claims.md Outdated
Comment thread docs/claims.md Outdated
@lsaether lsaether self-assigned this May 13, 2020
@joepetrowski joepetrowski removed the WIP label May 20, 2020
Comment thread docs/claims.md Outdated
Comment thread docs/claims.md Outdated
Comment thread docs/claims.md Outdated
Comment thread docs/claims.md Outdated
Comment thread docs/claims.md Outdated
Comment thread docs/claims.md Outdated
lsaether and others added 4 commits May 25, 2020 09:40
Co-authored-by: Bill Laboon <laboon@users.noreply.github.com>
Co-authored-by: Bill Laboon <laboon@users.noreply.github.com>
Co-authored-by: Bill Laboon <laboon@users.noreply.github.com>
Co-authored-by: Bill Laboon <laboon@users.noreply.github.com>
Comment thread docs/claims.md
Comment thread docs/claims.md Outdated
Comment thread docs/claims.md
Comment thread docs/claims.md Outdated
Comment thread docs/claims.md Outdated
Comment thread docs/claims.md Outdated
Comment thread docs/claims.md
Comment thread docs/claims.md Outdated
Comment thread docs/claims.md Outdated
Comment thread docs/claims.md Outdated
Comment thread docs/claims.md Outdated
Comment thread docs/claims.md Outdated
Comment thread docs/claims.md Outdated
Comment thread docs/claims.md Outdated
lsaether and others added 11 commits May 25, 2020 16:19
Co-authored-by: Bill Laboon <laboon@users.noreply.github.com>
Co-authored-by: Bill Laboon <laboon@users.noreply.github.com>
Co-authored-by: Bill Laboon <laboon@users.noreply.github.com>
Co-authored-by: Bill Laboon <laboon@users.noreply.github.com>
Co-authored-by: Bill Laboon <laboon@users.noreply.github.com>
@lsaether
Copy link
Copy Markdown
Contributor

Waiting on new screenshots from @ansonla3

Copy link
Copy Markdown
Contributor

@Swader Swader left a comment

Choose a reason for hiding this comment

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

Apart from some typos on screenshots (submitting separately to Apps) everything seems fine

Comment thread docs/claims.md Outdated
Comment thread docs/claims.md Outdated
Comment thread docs/claims.md Outdated
Comment thread docs/claims.md
prior sales. You will need to have access to this account in order to make a signature.

[MyCrypto][] is a versatile wallet that supports a variety of storage methods for your Ethereum
account. Go to their downloads page and ensure that you download the latest version for your
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Should we include a link to their downloads page here? Or say that it was linked previously?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

It is linked in the MyCrypto right there. But could just move that to "downloads page" in the text.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Yeah, it was linked in the previous sentence, but to me this just looks awkward saying "go to their downloads page" without a link or reference.

My preference would be to move the link to here.

Comment thread docs/claims.md Outdated
Comment thread docs/claims.md

The Polkadot JS [Claims app][] helps you sign a message from MyCrypto. MyCrypto is a good choice in
case you have stored the key to the Ethereum account holding your DOT indicator tokens on a hardware
device like a Ledger Nano S or a Trezor. It also supports raw private keys, mnemonics, and Parity
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

MyCrypto with Trezor can't sign, apparently. Not sure why.
Screen Shot 2020-05-26 at 4 38 57 PM

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I wonder if this is a new change?

Comment thread docs/claims.md Outdated
Comment thread docs/claims.md Outdated
lsaether and others added 6 commits May 26, 2020 19:25
Co-authored-by: Bill Laboon <laboon@users.noreply.github.com>
Co-authored-by: Bill Laboon <laboon@users.noreply.github.com>
Co-authored-by: Bill Laboon <laboon@users.noreply.github.com>
Co-authored-by: Bill Laboon <laboon@users.noreply.github.com>
Co-authored-by: Bill Laboon <laboon@users.noreply.github.com>
Co-authored-by: Bill Laboon <laboon@users.noreply.github.com>
@lsaether lsaether merged commit b816ddc into master May 26, 2020
@lsaether lsaether deleted the post-genesis-claim branch May 26, 2020 17:27
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.

Update Claiming for attesting to statements Update the KSM Claims page for DOT Claims

7 participants