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.

Sign up for GitHub

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

Jump to bottom

Dex Aggregator Feat: Lifi Optimism - Ready for review #4082

Merged
merged 14 commits into from
Aug 22, 2023
Merged

Dex Aggregator Feat: Lifi Optimism - Ready for review #4082

merged 14 commits into from
Aug 22, 2023

Conversation

henrystats
Copy link
Contributor

Thank you for contributing to Spellbook!

Please refer to the top of the readme in the root of Spellbook to learn how to contribute to Spellbook on DuneSQL.

@dune-eng
Copy link

Workflow run id 5863390304 approved.

@dune-eng
Copy link

Workflow run id 5863390346 approved.

@dune-eng
Copy link

Workflow run id 5863391560 approved.

@dune-eng
Copy link

Workflow run id 5863391554 approved.

@henrystats henrystats changed the title Dex Aggregator Feat: Lifi Optimism Dex Aggregator Feat: Lifi Optimism - Ready for review Aug 15, 2023
@henrystats
Copy link
Contributor Author

henrystats commented Aug 15, 2023 •
edited
Loading

This pr adds lifi aggregator trades on optimism.. I created this branch from the lifi_fantom migration pr I set up for migrating life on fantom.

@antonio-mendes antonio-mendes added dbt: dex covers the DEX dbt subproject DuneSQL migration labels Aug 17, 2023
@dune-eng
Copy link

Workflow run id 5906294734 approved.

@dune-eng
Copy link

Workflow run id 5906294769 approved.

@dune-eng
Copy link

Workflow run id 5906443190 approved.

@dune-eng
Copy link

Workflow run id 5906443254 approved.

@dune-eng
Copy link

Workflow run id 5930922033 approved.

@dune-eng
Copy link

Workflow run id 5930922150 approved.

@dune-eng
Copy link

Workflow run id 5931024728 approved.

@dune-eng
Copy link

Workflow run id 5931025013 approved.

@henrystats
Copy link
Contributor Author

apologies for another tag @jeff-dude 😅 -- this is a pr that's very similar to the fantom one that was merged earlier... since you were merging the other aggregator pr's.

I built this off the fantom one so it's ready for review/merging 😅

@jeff-dude jeff-dude merged commit 5926f1c into duneanalytics:main Aug 22, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Aug 22, 2023
@henrystats henrystats deleted the add-lifi-optimism branch August 23, 2023 03:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@jeff-dude jeff-dude jeff-dude approved these changes

Assignees
No one assigned
Labels
dbt: dex covers the DEX dbt subproject
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@henrystats @dune-eng @jeff-dude @antonio-mendes