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

Support alternative receiver feature in Monitor #223

Merged
merged 18 commits into from
Oct 29, 2019

Conversation

patitonar
Copy link
Contributor

Built on top of #221

The main changes to the monitor component were made on this commit: eef52e1

Included also a fix for the chromedriver used in e2e tests to avoid CI failing

@patitonar patitonar added the monitor related to TokenBridge monitor label Oct 24, 2019
@patitonar patitonar requested a review from akolotov October 24, 2019 19:24
@patitonar patitonar self-assigned this Oct 24, 2019
Copy link
Collaborator

@akolotov akolotov left a comment

Choose a reason for hiding this comment

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

LGTM. Do you think we need to wait for the merge of #221 before merging the changes for the monitor?

@patitonar
Copy link
Contributor Author

Yes, let's wait for #221 to be merged in case more changes are applied there. Then we can update this branch with develop and merge this changes

@patitonar
Copy link
Contributor Author

This is ready to be merged

@akolotov akolotov merged commit 1564ccc into develop Oct 29, 2019
@akolotov akolotov deleted the monitor-support-alternative-receiver branch October 29, 2019 19:22
akolotov added a commit that referenced this pull request Nov 12, 2019
…230

* Support alternative receiver in Oracle (#221)
* Support alternative receiver feature in Monitor (#223)
* Support token migration (#224)
* Fix suggested gas price in transaction for ui production build (#222)
* Updated links to new repo with tokenbridge contracts (#226)
* Update the contract's submodule to the release 3.2.0 (#228)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
monitor related to TokenBridge monitor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants