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

Remove timestamp filters in dex.trades enrichment #7363

Merged
merged 1 commit into from
Dec 20, 2024
Merged

Remove timestamp filters in dex.trades enrichment #7363

merged 1 commit into from
Dec 20, 2024

Conversation

aalan3
Copy link
Contributor

@aalan3 aalan3 commented Dec 20, 2024

A bug was introduced in #7350 where we forgot to remove the timestamp filters used during testing.

@github-actions github-actions bot marked this pull request as draft December 20, 2024 15:51
@github-actions github-actions bot added WIP work in progress dbt: dex covers the DEX dbt subproject labels Dec 20, 2024
@aalan3 aalan3 marked this pull request as ready for review December 20, 2024 15:53
@github-actions github-actions bot added ready-for-review this PR development is complete, please review and removed WIP work in progress labels Dec 20, 2024
@aalan3 aalan3 merged commit 6a2d09f into main Dec 20, 2024
3 of 4 checks passed
@aalan3 aalan3 deleted the hotfix-remove-timestamps branch December 20, 2024 16:02
@github-actions github-actions bot locked and limited conversation to collaborators Dec 20, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
dbt: dex covers the DEX dbt subproject ready-for-review this PR development is complete, please review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@aalan3