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

DUX-939 added new evm chains #7802

Merged
merged 7 commits into from
Mar 27, 2025
Merged

DUX-939 added new evm chains #7802

merged 7 commits into from
Mar 27, 2025

Conversation

krishgka
Copy link
Contributor

@krishgka krishgka commented Mar 6, 2025

Thank you for contributing to Spellbook 🪄

Please open the PR in draft and mark as ready when you want to request a review.

Description:

[...]


quick links for more information:

@krishgka krishgka requested a review from jeff-dude March 6, 2025 01:22
@github-actions github-actions bot marked this pull request as draft March 6, 2025 01:22
@github-actions github-actions bot added WIP work in progress dbt: tokens covers the Tokens dbt subproject labels Mar 6, 2025
@jeff-dude jeff-dude self-assigned this Mar 6, 2025
@jeff-dude jeff-dude marked this pull request as ready for review March 6, 2025 15:32
@github-actions github-actions bot added ready-for-review this PR development is complete, please review and removed WIP work in progress labels Mar 6, 2025
@jeff-dude jeff-dude added in review Assignee is currently reviewing the PR and removed ready-for-review this PR development is complete, please review labels Mar 6, 2025
@krishgka krishgka changed the title added new evm chains DUX-939 added new evm chains Mar 10, 2025
@jeff-dude jeff-dude marked this pull request as draft March 11, 2025 17:52
@jeff-dude jeff-dude removed the in review Assignee is currently reviewing the PR label Mar 11, 2025
@github-actions github-actions bot added the WIP work in progress label Mar 11, 2025
@jeff-dude
Copy link
Member

on pause until we finalize existing EVM chains into metrics pipeline

@jeff-dude
Copy link
Member

blast failure makes me second guess the assigned native token here -- we sure it's not an erc20 that is being accounted for in those transfers already? my guess is we already have that address in transfers

@krishgka krishgka added ready-for-review this PR development is complete, please review and removed WIP work in progress labels Mar 27, 2025
@krishgka krishgka marked this pull request as ready for review March 27, 2025 21:08
@jeff-dude jeff-dude removed the ready-for-review this PR development is complete, please review label Mar 27, 2025
@jeff-dude jeff-dude merged commit f4d513d into main Mar 27, 2025
5 checks passed
@jeff-dude jeff-dude deleted the new_evm_net_transfers branch March 27, 2025 21:38
@github-actions github-actions bot locked and limited conversation to collaborators Mar 27, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
dbt: tokens covers the Tokens dbt subproject ready-for-merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants