Skip to content

Conversation

@GBKS
Copy link
Contributor

@GBKS GBKS commented Dec 7, 2021

Initial commit with fairly rough content. Still needs a ton of refinement, so keeping this is a draft for now. So it's not ready for review quite yet, but I am creating the PR so I can get direct feedback on some specific areas.

🚀Preview🧨

The second new page is a sub-page to "Sending bitcoin", and called "Payment request formats". Preview it here.

Initial commit with fairly rough content. Still needs a ton of refinement.
@GBKS GBKS added payments Copy Task is about improving text. Design Task is about designing something. labels Dec 7, 2021
@GBKS GBKS added this to the Milestone #11 milestone Dec 7, 2021
@GBKS GBKS self-assigned this Dec 7, 2021
@GBKS GBKS linked an issue Dec 7, 2021 that may be closed by this pull request
@pavlenex pavlenex removed this from the Milestone #11 milestone Dec 8, 2021
@GBKS GBKS changed the title Sending bitcoin page Lightning addition Added ⚡️ content to Payments > Sending bitcoin Dec 13, 2021
@GBKS GBKS marked this pull request as ready for review December 13, 2021 15:00
Copy link
Contributor

@pavlenex pavlenex left a comment

Choose a reason for hiding this comment

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

Nice work and lots of screens which makes me so excited :) I assume that this also touches #580 and does it address all @moneyball's points in #560?

GBKS and others added 8 commits December 16, 2021 12:07
Co-authored-by: bosch <55287964+Bosch-0@users.noreply.github.com>
Co-authored-by: bosch <55287964+Bosch-0@users.noreply.github.com>
Co-authored-by: bosch <55287964+Bosch-0@users.noreply.github.com>
Co-authored-by: bosch <55287964+Bosch-0@users.noreply.github.com>
Co-authored-by: bosch <55287964+Bosch-0@users.noreply.github.com>
Co-authored-by: bosch <55287964+Bosch-0@users.noreply.github.com>
- Added image that shows editing fees
- Capitalized payment format examples
- Switched gallery to include
- Added home screen image to clarify manual payment initiation flow
- Rewrite "address" bit and changed to "recipient"
- Added "meta data" description
The need to send bitcoin can be triggered by many different use cases, and initiated by both sender and receiver. Common entry points are:

You do not need to follow the order below. Feel free to tailor the configuration's order for the payment to what best serves your users. For example, you may make users set the amount before they enter the address.
- Manual entry of address and amount
Copy link
Contributor

Choose a reason for hiding this comment

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

What's the difference between:

  • Manual entry of address and amount and
  • Copy & pasting payment information?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Manual entry is the most basic entry form and has to be supported.

Copy & paste can be an interaction with an input field, but ideally, the app auto-detects clipboard content and presents the appropriate UI automatically.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Does it have to be supported though? I can't imagine entering it like that and if they did it has major room for error.

Copy link
Collaborator

@Bosch-0 Bosch-0 left a comment

Choose a reason for hiding this comment

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

ACK from me :)

Copy link
Contributor

@pavlenex pavlenex left a comment

Choose a reason for hiding this comment

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

LGTM for a first, there may be more things we discover as we go forward in 2022. Good work Christoph!

Sending bitcoin is one of the most essential user activities in a Bitcoin application, and also one of the least structured ones. People may want to send bitcoin to a known contact, transfer it to another wallet on a different device, or make a purchase via a [payment processor]({{ '/guide/getting-started/software/#payment-processors' | relative_url }}).

Let us look at the main transaction options senders need to configure when moving bitcoin:
## Payment entry points
Copy link
Collaborator

Choose a reason for hiding this comment

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

Another payment entry point you may want to consider adding is generating and sharing a withdraw request. I'm covering this in my requesting bitcoin page. By withdraw request I'm referring to LNURL-withdraw (withdraw request sounds better imo).

You could expand on this somewhere in this section and discuss how these could be used to generate P2P gift cards.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I was thinking we could have a "Withdrawing bitcoin" page at a later point.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Would there be enough content for another dedicated page? Could easily be covered across sending / request page imo

Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think it would be enough for a dedicated page. It's def a use-case we should cover, perhaps in a follow up? One more example besides a gift would be a user getting a refund from a merchant so they need to scan a QR in order to pull the funds out.

@GBKS
Copy link
Contributor Author

GBKS commented Dec 21, 2021

Thanks for the feedback everyone.

@pavlenex 100% sure we need to tweak things, especially with the follow-up task to smoothen out bumps between pages once all our current PRs have been merged.

Copy link
Collaborator

@sbddesign sbddesign left a comment

Choose a reason for hiding this comment

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

LGTM

@pavlenex
Copy link
Contributor

Merging this one, since it's important foundation for the upcoming receive PR and also since it's been ack'd a few times.

There are two remaining comments left about LNURL here and manual amount entry here which we agreed may be suitable for smaller follow up PR's after we have a consensus on how we may want to proceed.

Great work Christoph and everyone involved, this was a big challenge, let's go! 🚀

@pavlenex pavlenex merged commit d5e04ed into master Dec 21, 2021
@danielnordh danielnordh deleted the feature/sending-bitcoin-lightning-iteration branch February 10, 2022 12:31
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. Design Task is about designing something.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add ⚡️ content to Payments > Sending bitcoin Add guidelines for LN sent payment confirmation UX

5 participants