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

Update dex.trades with Elkfinance #7289

Merged
merged 7 commits into from
Dec 10, 2024
Merged

Update dex.trades with Elkfinance #7289

merged 7 commits into from
Dec 10, 2024

Conversation

blanchemaria6
Copy link
Contributor

Description:

Update dex.trades with Elkfinance on Gnosis

quick links for more information:

@Hosuke Hosuke added the dbt: dex covers the DEX dbt subproject label Dec 10, 2024
Hosuke
Hosuke approved these changes Dec 10, 2024
View reviewed changes
Copy link
Collaborator

@Hosuke Hosuke left a comment •
edited
Loading

Choose a reason for hiding this comment

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

LGTM.✅
Thank you @blanchemaria6

Sample elk_finance_gnosis_base_trades

@jeff-dude jeff-dude self-assigned this Dec 10, 2024
@jeff-dude jeff-dude merged commit cd9372c into duneanalytics:main Dec 10, 2024
2 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Dec 10, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@Hosuke Hosuke Hosuke approved these changes

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

Assignees

@jeff-dude jeff-dude

Labels
dbt: dex covers the DEX dbt subproject ready-for-merging
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@blanchemaria6 @Hosuke @jeff-dude