BTCPay Server 2.0: Megathread about btcpayserver

[Kukks]

API: Permission on Refund Invoice endpoint The current permission is btcpay.store.cancreatepullpayments but it should imo be btcpay.store.cancreatenonapprovedpullpayments

I believe we can do this without a breaking change as it softens the permission

[NicolasDorier]

Refactor onchain support to be similar to LN (connection string style, allows plugins to provide their own onchain tracker system instead)

@Kukks I believe it isn't possible. We are querying NBX database directly in SQL.

[pavlenex]

Convert various UIs to Blazor

Can we be more specific when listing issues, what does this mean?

[Kukks]

Convert various UIs to Blazor

Can we be more specific when listing issues, what does this mean?

I believe it is any UI that uses Vue currently. We can work on a list later

[pavlenex]

Alright, I understood this list should be a list of breaking changes, but it seems we're pilling quite a lot of things here, and I am trying to understand how to prioritize what is now becoming quite a massive list of tasks.

[ndeet]

2nd try, had written everything up got distracted and shut down the laptop, all gone 🙈

I have no access to tasklist so posting here for review and discussion.

API: inconsistent webhook event naming (breaking change)
We have InvoiceReceivedPayment and InvoicePaymentSettled, it would be more logical to have them aligned and change InvoiceReceivedPayment to InvoicePaymentReceived

API: Clean up store wallet docs
We have two sections in the navigation "Store Payment Methods (On Chain)" (SPM) and "Store Wallet (On Chain)" (SW), both use the same endpoint path (stores/{storeId}/payment-methods/onchain/{cryptoCode}/) and the listed endpoints are mixed up a bit.

1. SPM has the Create store on-chain wallet endpoint listed which better fits to SW
2. The response of above create wallet endpoint contains a legacy field which is not settable through above mentions endpoint called label that is always null on newer wallets
3. Discuss: Preveiw store on-chain payment method addresses and Preview proposed store on-chain payment methods should also be renamed and moved to SW as it relates to wallet and not payment method, if there is a reason to have it here then maybe move Get Store onchain-wallet address to have address fetching together in one section

API: Cleanup Apps / Crowdfund / PoS (possibly breaking change)

1. The "App" section has create/create/delete endpoints for both Crowdfund and PoS but their specific sections only 1-2 endpoints, either move everything to apps or to their specific sections
2. Update Crowdfund endpoint is missing (at least in docs)
3. there is only one general Delete App endpoint but not specific for crowdfund or PoS

I had a few other API related issues on my list but those inconsistencies have been already fixed.

Does anybody find the old BTCPay 2.0 megathread, afair there was one but I could not find it?

[ndeet]

API: Permission on Refund Invoice endpoint
The current permission is btcpay.store.cancreatepullpayments but it should imo be btcpay.store.cancreatenonapprovedpullpayments

[ndeet]

API: Invoice refund inconsistency
With the pull payments endpoint you can set paymentMethods as an array and if not set it will apply all available payment methods, with the invoice refund endpoint you only can pass a string and it is required too.

Suggestion: Make it a nullable array too to be consistent and more flexible