Skip to content

Conversation

@wbobeirne
Copy link
Member

Closes #79. Will want a review from @NicolasDorier on this one.

Description

Adds an option to the node selector for BTCPay Server. Uses the BTCPay Server config format for filling out URI & macaroons.

Steps to Test

  1. Setup a BTCPay Server
  2. Go through Joule onboarding, select BTCPay Server
  3. Follow the instructions and fill out the config data
  4. Confirm you connect correctly

Screenshots

onboarding

^Click to embiggen

@NicolasDorier
Copy link

This is awesome: Though can you do this:

In the configuration file, right now you can find the macaroon.
In a future version, I will add the readonlymacaroon.

Can you, in your code consider that if readonlymacaroon is null or does not exist, then use macaroon for both?

That way, when I update BTCPay it will properly use the right one.

private handleBTCPayServerConfig = (config: BTCPayServerConfig) => {
const macaroons = {
adminMacaroon: config.macaroon,
readonlyMacaroon: config.restrictedMacaroon || config.macaroon,

Choose a reason for hiding this comment

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

use readonlyMacaroon instead of restrictedMacaroon

@NicolasDorier
Copy link

NicolasDorier commented Dec 16, 2018

Ah actually you do this, just rename restrictedMacaroon to readonlyMacaroon.
I made a wrong choice.

@wbobeirne
Copy link
Member Author

Thanks for that, the change has been made. I'll be merging shortly.

@wbobeirne wbobeirne merged commit 7b90499 into develop Dec 18, 2018
@wbobeirne wbobeirne deleted the btc-pay-server-option branch December 18, 2018 03:48
wbobeirne pushed a commit that referenced this pull request Oct 13, 2020
…/typescript-3.9.6

Update typescript: 3.9.3 → 3.9.6 (patch)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants