Skip to content

added bullet point on wallet fees transparency #538

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

Merged

Conversation

nguyener1
Copy link
Contributor

See #536 for a discussion of changes.

Made a copy edit that combines both suggestions in the issue tracker.

@GBKS GBKS added the Copy Task is about improving text. label Oct 25, 2021
@GBKS GBKS linked an issue Oct 26, 2021 that may be closed by this pull request
Copy link
Contributor

@GBKS GBKS left a comment

Choose a reason for hiding this comment

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

Great tweak, thank you for submitting.

Only addition I could think of is the term "pro-active". It's the worst when fees are hidden in small text somewhere or they tell you too late and you wouldn't have take your current course of action if you had known about fees earlier.

@pavlenex
Copy link
Contributor

@GBKS Good point about pro-active and transparent fee display.
Any page suggestion where this can be added?
I think it that can be added to a mockup/image rather than the principles section? If so, we can merge this and add a note to add a bit more regarding pro-active fee display.

@Bosch-0
Copy link
Collaborator

Bosch-0 commented Oct 27, 2021

I think it that can be added to a mockup/image rather than the principles section?

Agree with this, we haven't given examples to other sections of the principles page, why give one here? Better to apply those principles in context. For example in the Funding a wallet page I am working on I have a few pointers about fee transparency.

@pavlenex
Copy link
Contributor

In that case, I'll go ahead and merge this one and we'll proceed with adding this in the funding a wallet, @Bosch-0 <3

@pavlenex pavlenex merged commit ba54306 into BitcoinDesign:master Oct 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Copy Task is about improving text.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Fee Transparency in the Principles don's and don't section
4 participants