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 Contracts and Events #3359

Merged
merged 4 commits into from May 5, 2022
Merged

Conversation

rdig
Copy link
Member

@rdig rdig commented May 4, 2022

This PR updates colonyJS to support the newly released contract versions, as well as update all the related submodules that are relevant to this (colonyNetwork and subgraph):

  • Colony v9
  • OneTx v3
  • Coin Machine v4 and v5 (defaults to the latest)
  • Voting Reputation v4
  • Whitelist v2

This also updates the SubgraphEventsThatAreActions subscription so that it tracks the both the old TokenUnlocked() and the new TokenUnlocked(address) events that were added to the contract

I've also updated the subgraph repo separately so that it tracks (and also supports) the new events.

Note that the rc release of colonyJS is temporary, and once I confirm it works on QA, will release a mainline version and will update this PR with it.

@rdig rdig self-assigned this May 4, 2022
@rdig rdig requested a review from a team May 4, 2022 12:21
@rdig rdig force-pushed the maintenance/update-contract-versions branch from c0c746a to 5b0fb70 Compare May 4, 2022 12:23
Copy link
Contributor

@chinins chinins left a comment

Choose a reason for hiding this comment

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

The most time it took for the review was actually to rebuild the environment :)

@rdig rdig force-pushed the maintenance/update-contract-versions branch from 5b0fb70 to 8a6332a Compare May 5, 2022 13:22
@rdig rdig merged commit 9c0282c into master May 5, 2022
@rdig rdig deleted the maintenance/update-contract-versions branch May 5, 2022 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants