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
Extract send fees into a page and add scenarios #1025
Conversation
Picking up work that Steyn did around various fee scenarios, and tried to marry it with what we have in the guide already. The "Send" page had a lot of fee-related info, but adding more there would have resulted in a very long page. That's why I extracting the fee content into a dedicated page that lives underneath the "Send" page. Closes #997
✅ Deploy Preview for bitcoin-design-site ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work Easy to read and understand language.
assets/images/guide/daily-spending-wallet/sending/send-fees/fee-options-custom-options-big.png
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very easy to ready and understand, I really like the visuals as well. Added in some minor text suggestions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good! I agree with the content. However, I found a typo, a questionable phrasing, and a new image file that is not referenced within the document.
Co-authored-by: Mogashni <90313394+mouxdesign@users.noreply.github.com> Co-authored-by: Stephen DeLorme <stephen@stephendelorme.com>
Thank you for your feedback, dear reviewers. I accepted it all, please take another look and leave your verdict. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Looks good to 🚀 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✌️
Picking up work that Steyn did around various fee scenarios, and tried to marry it with what we have in the guide already. The "Send" page had a lot of fee-related info, but adding more there would have resulted in a very long page. That's why I extracted the fee content into a dedicated page that lives underneath the "Send" page. The "Send" page now only has a very short paragraph about fees and links out to this new page.
Some content changes:
🧑🏻🏭Check the preview👨🏼🏭
Closes #997