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

Automatically define Block reward contract address in TokenBridge supply module #3064

Merged
merged 1 commit into from
Apr 2, 2020

Conversation

vbaranov
Copy link
Member

@vbaranov vbaranov commented Apr 2, 2020

Motivation

Block reward contract address is hardcoded in TokenBridge supply module

Changelog

Extract Block reward contract address from blockRewardContract property of TokenBridge contract

Checklist for your Pull Request (PR)

@vbaranov vbaranov changed the title Automatically define Block reward contract address inTokenBridge supply module Automatically define Block reward contract address in TokenBridge supply module Apr 2, 2020
@vbaranov vbaranov force-pushed the vb-token-bridge-block-reward branch from ada6c81 to 9bd59f9 Compare April 2, 2020 09:28
@coveralls
Copy link

coveralls commented Apr 2, 2020

Pull Request Test Coverage Report for Build a49649e4-e9e8-4694-955c-012e7665a469

  • 0 of 10 (0.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.06%) to 74.912%

Changes Missing Coverage Covered Lines Changed/Added Lines %
apps/explorer/lib/explorer/chain/supply/token_bridge.ex 0 10 0.0%
Totals Coverage Status
Change from base Build 59bed80f-f472-496d-9262-32233ab43b5c: -0.06%
Covered Lines: 5515
Relevant Lines: 7362

💛 - Coveralls

@vbaranov vbaranov force-pushed the vb-token-bridge-block-reward branch from 9bd59f9 to 3d3cbb5 Compare April 2, 2020 09:43
@vbaranov vbaranov merged commit f9be788 into master Apr 2, 2020
@vbaranov vbaranov deleted the vb-token-bridge-block-reward branch April 2, 2020 10:34
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.

None yet

2 participants