Skip to content
This repository was archived by the owner on Apr 17, 2023. It is now read-only.

Fixes #1964 Adding notice to wallet about Stacks Wallet - #1965

Merged
hstove merged 3 commits into
stacks-archive:developfrom
moxiegirl:wallet-note
Nov 13, 2019
Merged

Fixes #1964 Adding notice to wallet about Stacks Wallet#1965
hstove merged 3 commits into
stacks-archive:developfrom
moxiegirl:wallet-note

Conversation

@moxiegirl

@moxiegirlmoxiegirl commented Oct 30, 2019

Copy link
Copy Markdown

This PR implements the following:

image

Signed-off-by: Mary Anthony mary@blockstack.com

Signed-off-by: Mary Anthony <mary@blockstack.com>
@moxiegirlmoxiegirl changed the title Fixes #253 Adding notice to wallet about Stacks WalletFixes #1964 Adding notice to wallet about Stacks WalletOct 30, 2019
Signed-off-by: Mary Anthony <mary@blockstack.com>

@markmhendricksonmarkmhendrickson left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Looks great!

@moxiegirl

moxiegirl commented Nov 5, 2019

Copy link
Copy Markdown
Author

Changed text to this:

image

Signed-off-by: Mary Anthony <mary@blockstack.com>
@moxiegirl

Copy link
Copy Markdown
Author

@hstove assigned you to this for the merge/build step

@hstove

Copy link
Copy Markdown
Contributor

This is great! Thanks, @moxiegirl

@hstove
hstove merged commit c9257da into stacks-archive:developNov 13, 2019
<div>
<Navbar activeTab="wallet" />
<div className="container-fluid col-centered form-container-secondary">
<strong>NOTE:</strong> You cannot use this wallet to send and receive Stacks (STX) tokens. Also, you cannot use the Bitcoin (BTC) address on this page to fund STX transactions. This wallet and its address <strong>only</strong> support the purchase of Blockstack identities(IDs). <strong>To create or fund STX transactions, use the Stacks Wallet software.</strong> See <a href="https://docs.blockstack.org/org/wallet-install.html" target="_blank">the Stacks Wallet software documentation</a> for more information.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This PR has been merged but I spot the need for a space here before "(IDs)" so @moxiegirl or @hstove you may want to create a small new PR to correct.

@hstovehstove mentioned this pull request Nov 19, 2019
Sign up for freeto 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

@moxiegirl@hstove@markmhendrickson