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

[FOR REVIEW] Add ZeroEx Settler trades on Base #6413

Merged
merged 107 commits into from
Jul 30, 2024

Conversation

RantumBits
Copy link
Contributor

Thank you for contributing to Spellbook 🪄

Contribution type

Please check the type of contribution this pull request is for:

  • New spell(s)
  • Adding to existing spell lineage
  • Bug fix

Note: You can safely discard any section below which doesn't apply based on selection above


For new spell(s)

If you are building new spell(s), please provide the following information:

  • Spell name(s): schema.table_name zeroex_base.settler_trades
  • Description: [Detailed description of the new spell(s) and their purpose] add dex agg trades from new Settler contracts
  • Who are the new spell(s) for? [Internal team or general community] both
  • How will the new spell(s) be used downstream? [Description of downstream usage] added to dex_aggregator.trades
  • Implementation details: [Information on how the spell(s) are implemented] gets taker and maker amounts from logs, identifies txns via traces. no FIlls level data
  • Test instructions: [How to test the new spell(s)]
  • Related issue(s): [Link to related issues, if any] https://dune.com/queries/3921632

RantumBits and others added 30 commits June 1, 2021 14:07
@dune-eng
Copy link

Workflow run id 10084367794 approved.

@dune-eng
Copy link

Workflow run id 10084368015 approved.

@dune-eng
Copy link

Workflow run id 10084368018 approved.

@dune-eng
Copy link

Workflow run id 10101253477 approved.

@dune-eng
Copy link

Workflow run id 10102358677 approved.

@dune-eng
Copy link

Workflow run id 10102358825 approved.

@dune-eng
Copy link

Workflow run id 10102358823 approved.

@RantumBits
Copy link
Contributor Author

updated to match requested changes on #6425

Copy link
Collaborator

@Hosuke Hosuke left a comment

Choose a reason for hiding this comment

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

@Hosuke Hosuke added ready-for-final-review and removed ready-for-review this PR development is complete, please review labels Jul 29, 2024
@Hosuke Hosuke assigned jeff-dude and unassigned Hosuke Jul 29, 2024
@jeff-dude jeff-dude merged commit 5132afa into duneanalytics:main Jul 30, 2024
1 of 2 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Jul 30, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
dbt: dex covers the DEX dbt subproject ready-for-merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants