Skip to content

Tests for GET /v5/channel/list and multichain support#479

Merged
simzzz merged 9 commits intoaip-61-adex-v5from
channel-list-tests
Mar 28, 2022
Merged

Tests for GET /v5/channel/list and multichain support#479
simzzz merged 9 commits intoaip-61-adex-v5from
channel-list-tests

Conversation

@simzzz
Copy link
Contributor

@simzzz simzzz commented Mar 16, 2022

No description provided.

@simzzz simzzz requested a review from elpiel March 16, 2022 12:35
Copy link
Contributor

@elpiel elpiel left a comment

Choose a reason for hiding this comment

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

Minor things only, good job 😋

@simzzz simzzz requested a review from elpiel March 24, 2022 13:01
@elpiel elpiel mentioned this pull request Mar 24, 2022
21 tasks
@elpiel elpiel changed the title Tests for GET /v5/channel/list Tests for GET /v5/channel/list and multichain support Mar 24, 2022
Copy link
Contributor

@elpiel elpiel left a comment

Choose a reason for hiding this comment

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

  • Fix the conflicting file
  • One test is failing because we've added the 0x000 address to config, you can change it to 00..1 until I deploy a new contract on a new Chain Id in ganache.

@simzzz simzzz requested a review from elpiel March 28, 2022 13:45
@simzzz simzzz merged commit 3aa3346 into aip-61-adex-v5 Mar 28, 2022
@elpiel elpiel deleted the channel-list-tests branch March 28, 2022 15:10
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.

2 participants