Closed Zaxounette closed 3 years ago
Proposing agenda, fill free to add topics.
Quick item: btcpayserver/btcpayserver#2915
Other than that it's just the LNURL UI discussion brought up on Mattermost yesterday from my side. Copying it and kukks' answers to the questions here for better overview.
Each of those options brings up questions that would need answering and to be honest, even I don't fully understand most of them.
political agenda i guess. But no.
Classic mode is the current way of representing lnurl. This is LNURL1blablabla - a bech32 encoding with lnurl as the hrp with its contents being the lnurl endpoint url. There is a new proposal that changes from this to a much better format: a cleartext url but with the scheme changed to whicever lnurl protocol is being used. For example: lnurlp:kukks.btcpay.org/i/invoiceid
Because a user might be confused seeing both bolt11 AND lnurl pay payment options. When there is a bolt 11 invoice already generated, it is wasteful to provide a way to generate more invoices. That said, some of the cooler features such as lightning address, pay button and apps support will REQUIRE this to be on
Because it is wasteful to generate a bolt11 invoice and monitor that and then offer another payment method that generates even more bolt 11 invoices that need to be monitored (ln nodes are not as rock solid as we'd want to just mindlessly have them monitor thousands of bolt11 invoices) THIS CRITIQUE IS ONLY VALID FOR WHEN LAZY PAYMENTS = FALSE
Nope!, As long as the connected ln node supports the featureset that's needed, you are good to use lnurl with it!
My take: I'd really like to strive for simplicity here, because I think this gets confusing really fast. We can always extend the set of options if there's a need for it, but doing so preemptively also means we have to document the questions/differences mentioned above.
Proposed Agenda (sorry planned to post way earlier, but had migrane and had to try to rest), some of these are duplicates of @pavlenex's, so we'll just move on to the others listed:
Store Focused Views Discussion (Figma)
Pull Payments Description (assign): https://github.com/btcpayserver/btcpayserver/issues/2625
Improve Rescan wording:
Refund flow:
Discuss any actions for: https://github.com/btcpayserver/btcpayserver/pull/2804
Updates on: https://github.com/btcpayserver/btcpayserver/pull/2878
Total Balance Updates? https://github.com/btcpayserver/btcpayserver/discussions/2306#discussioncomment-1336629 & https://github.com/btcpayserver/btcpayserver/pull/2882
Any additional thoughts...seems pretty straightforward, but worth mentioning: https://github.com/btcpayserver/btcpayserver/discussions/2911
Design Call N°19 notes Attendees: Rockstar, Zaxounette, Pavlenex, Dstrukt, Connor, Dennis
Last week we talked about empty state improvements and how we handle situations where users are new and have no store or wallet. We spoke about having a store centric idea and a pre-created store upon first setup of BTCPay Server or first user connexion.
We also discussed bringing this store centric idea to every view we have. Dstrukt presents the different approaches to these store centric views.
Discussing a pattern of swiping left/right to get access to the top menu items on mobile
The general direction this is going is good. One thing we're missing though is the meta hierarchy data.
We discuss the invoices page being visible on a higher level that aggregates the invoices of all stores for easy retrieval and management.
Discussing creating a store on first connexion of a user
LNURL discussion
Kukks's PR has many areas we can talk about:
General update:
Agenda
Comment below to add items to the agenda.
Check your timezone
https://everytimezone.com/s/4559e8e7
Join the call
https://meet.jit.si/WeDaBestCrewEVA
Calendar invite
Subscribe to the BTCPay Server calendar. More info here.