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

Use ER in EVM CE 2 #10032

Open
wants to merge 21 commits into
base: master
Choose a base branch
from
Open

Use ER in EVM CE 2 #10032

wants to merge 21 commits into from

Conversation

timolegros
Copy link
Collaborator

@timolegros timolegros commented Nov 26, 2024

Link to Issue

Closes: #10015

Description of Changes

  • Updated EvmEventSources table
    • Added contract_name and parent_contract_address which are used to map records to an event source in the EventRegistry (see childContracts).
    • Removed links to chain_node_id (now eth_chain_id) and abi_id.
  • Added a migration which removes all factory (stable/self-deployed) contracts (now live in the EventRegistry) leaving only the child contracts (see above).
    • Currently, the only child contracts are the contest contracts which include: started, voted, and content added events
  • Modified getEventSources to use the EventRegistry + child contracts (contracts launched by users) in the EvmEventSources table
  • Removed Contracts and ContractAbis tables/models

Test Plan

  • Launch stake + contests then interact with them

Deployment Plan

Other Considerations

# Conflicts:
#	libs/model/src/models/contract_abi.ts
#	libs/model/src/models/evmEventSource.ts
#	libs/model/test/contest-worker/contest-worker-policy.spec.ts
#	libs/model/test/contest/contests-projection-lifecycle.spec.ts
#	libs/schemas/src/entities/contract.schemas.ts
#	packages/commonwealth/test/devnet/evm/evmChainEvents.spec.ts
#	packages/commonwealth/test/integration/evmChainEvents/getEventSources.spec.ts
#	packages/commonwealth/test/integration/evmChainEvents/util.ts
# Conflicts:
#	libs/model/test/contest/check-contests.spec.ts
#	libs/model/test/contest/contest-worker-policy-lifecycle.spec.ts
@timolegros timolegros marked this pull request as ready for review November 27, 2024 00:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants