Skip to content
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

Update BTC config to use the correct domain #875

Merged
merged 1 commit into from Jun 11, 2019

Conversation

2 participants
@massao
Copy link
Contributor

commented Jun 7, 2019

What was the bug or feature?

URLs of connection to BTC bridge wasn't update to the correct domain.

How did I fix it?

Updated btc.config.js file to have the corrects domains.

Type of change

  • Bug fix

How to test it?

BTC should be able to connect after this fix.

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

@massao massao self-assigned this Jun 7, 2019

@massao massao marked this pull request as ready for review Jun 7, 2019

@reyraa reyraa self-requested a review Jun 11, 2019

@reyraa

reyraa approved these changes Jun 11, 2019

Copy link
Member

left a comment

Looks good

@reyraa reyraa added the ready label Jun 11, 2019

@reyraa reyraa added this to Pull Requests in 1.2.0 via automation Jun 11, 2019

@massao massao merged commit e32f3b0 into development Jun 11, 2019

2 checks passed

continuous-integration/jenkins/pr-merge This commit looks good
Details
coverage/coveralls First build on PR-875 at 94.227%
Details

1.2.0 automation moved this from Pull Requests to Merged Pull Requests Jun 11, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.