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

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

Merged
merged 3 commits into from
Nov 13, 2019
Merged

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

merged 3 commits into from
Nov 13, 2019

Conversation

moxiegirl
Copy link
Contributor

@moxiegirl moxiegirl commented Oct 30, 2019

This PR implements the following:

image

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

Signed-off-by: Mary Anthony <mary@blockstack.com>
@moxiegirl moxiegirl changed the title Fixes #253 Adding notice to wallet about Stacks Wallet Fixes #1964 Adding notice to wallet about Stacks Wallet Oct 30, 2019
Signed-off-by: Mary Anthony <mary@blockstack.com>
Copy link

@markmhendrickson markmhendrickson 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!

@moxiegirl
Copy link
Contributor Author

moxiegirl commented Nov 5, 2019

Changed text to this:

image

Signed-off-by: Mary Anthony <mary@blockstack.com>
@moxiegirl
Copy link
Contributor Author

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

@hstove
Copy link
Collaborator

hstove commented Nov 13, 2019

This is great! Thanks, @moxiegirl

@hstove hstove merged commit c9257da into stacks-archive:develop Nov 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.

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.

@hstove hstove mentioned this pull request Nov 19, 2019
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