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

Add gridex on polygon to dex.trades #6143

Merged
merged 6 commits into from
Jun 18, 2024

Conversation

blanchemaria6
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

For adding to existing spell lineage

  • Description: Add gridex on polygon to dex.trades

@dune-eng
Copy link

Workflow run id 9476834133 approved.

@dune-eng
Copy link

Workflow run id 9476834235 approved.

@dune-eng
Copy link

Workflow run id 9493707064 approved.

@dune-eng
Copy link

Workflow run id 9493707153 approved.

@dune-eng
Copy link

Workflow run id 9493721643 approved.

@dune-eng
Copy link

Workflow run id 9493721745 approved.

@jeff-dude jeff-dude requested a review from Hosuke June 17, 2024 17:36
@jeff-dude jeff-dude added ready-for-review this PR development is complete, please review dbt: dex covers the DEX dbt subproject labels Jun 17, 2024
@jeff-dude
Copy link
Member

based on merge conflicts, this will likely need rebased to fit into new dex subproject that was merged. @Hosuke can you help?

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 Jun 17, 2024
Copy link
Member

@jeff-dude jeff-dude left a comment

Choose a reason for hiding this comment

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

thanks for the help, hosuke!

@jeff-dude jeff-dude merged commit ae2d873 into duneanalytics:main Jun 18, 2024
1 of 3 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Jun 18, 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