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

feat: expose paraswap v6 decoded trades #7359

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

alexshchur
Copy link
Contributor

Description:

Expose ParaSwap v6 decoded trades

@github-actions github-actions bot marked this pull request as draft December 19, 2024 14:44
@github-actions github-actions bot added WIP work in progress dbt: dex covers the DEX dbt subproject labels Dec 19, 2024
@alexshchur alexshchur marked this pull request as ready for review December 19, 2024 15:04
@github-actions github-actions bot added ready-for-review this PR development is complete, please review and removed WIP work in progress labels Dec 19, 2024
@Hosuke Hosuke self-assigned this Dec 19, 2024
Comment on lines 10 to 13
post_hook='{{ expose_spells(\'["arbitrum"]\',
"project",
"paraswap_v6",
\'["eptighte", "mwamedacen"]\') }}'
Copy link
Collaborator

Choose a reason for hiding this comment

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

We can apply the new format for post_hooks:

Suggested change
post_hook='{{ expose_spells(\'["arbitrum"]\',
"project",
"paraswap_v6",
\'["eptighte", "mwamedacen"]\') }}'
post_hook='{{ expose_spells(blockchains = \'["arbitrum"]\',
spell_type = "project",
spell_name = "paraswap_v6",
contributors = \'["eptighte", "mwamedacen"]\') }}'

Copy link
Contributor Author

Choose a reason for hiding this comment

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

hey @Hosuke ,
i've tried to apply this new verbose format but I'm getting this compilation error now
Screenshot 2024-12-20 at 10 30 10

I've checked configs and we're not using the deprecated field "tests" and use "data_tests" instead.
Do you know what might be the cause?

@Hosuke Hosuke added in review Assignee is currently reviewing the PR and removed ready-for-review this PR development is complete, please review labels Dec 19, 2024
@alexshchur alexshchur requested a review from Hosuke December 20, 2024 08:32
@0xRobin
Copy link
Collaborator

0xRobin commented Dec 20, 2024

please merge in main to resolve the compile issue

@alexshchur
Copy link
Contributor Author

please merge in main to resolve the compile issue

That worked, thank you.

Is all good to merge then?

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.

LGTM.✅
Thank you @alexshchur

@Hosuke Hosuke added ready-for-merging and removed in review Assignee is currently reviewing the PR labels Dec 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dbt: dex covers the DEX dbt subproject ready-for-merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants