Skip to content

feat(Boost): add complete action to Boost plugin #414

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

Merged
merged 13 commits into from
May 28, 2024

Conversation

jonathandiep
Copy link
Contributor

No description provided.

Copy link

changeset-bot bot commented May 26, 2024

🦋 Changeset detected

Latest commit: 0f3a4f1

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 48 packages
Name Type
@rabbitholegg/questdk-plugin-registry Patch
@rabbitholegg/questdk-plugin-boost Patch
@rabbitholegg/questdk-plugin-utils Patch
@rabbitholegg/questdk-plugin-across Patch
@rabbitholegg/questdk-plugin-aerodrome Patch
@rabbitholegg/questdk-plugin-artblocks Patch
@rabbitholegg/questdk-plugin-balancer Patch
@rabbitholegg/questdk-plugin-base Patch
@rabbitholegg/questdk-plugin-basepaint Patch
@rabbitholegg/questdk-plugin-camelot Patch
@rabbitholegg/questdk-plugin-connext Patch
@rabbitholegg/questdk-plugin-fabric Patch
@rabbitholegg/questdk-plugin-gmx Patch
@rabbitholegg/questdk-plugin-handlefi Patch
@rabbitholegg/questdk-plugin-hop Patch
@rabbitholegg/questdk-plugin-hyphen Patch
@rabbitholegg/questdk-plugin-jojo Patch
@rabbitholegg/questdk-plugin-kote Patch
@rabbitholegg/questdk-plugin-kwenta Patch
@rabbitholegg/questdk-plugin-llama Patch
@rabbitholegg/questdk-plugin-manifold Patch
@rabbitholegg/questdk-plugin-mirror Patch
@rabbitholegg/questdk-plugin-mux Patch
@rabbitholegg/questdk-plugin-neynar Patch
@rabbitholegg/questdk-plugin-okutrade Patch
@rabbitholegg/questdk-plugin-optimism Patch
@rabbitholegg/questdk-plugin-orbit Patch
@rabbitholegg/questdk-plugin-paragraph Patch
@rabbitholegg/questdk-plugin-paraswap Patch
@rabbitholegg/questdk-plugin-pendle Patch
@rabbitholegg/questdk-plugin-pods Patch
@rabbitholegg/questdk-plugin-polygon Patch
@rabbitholegg/questdk-plugin-rabbithole Patch
@rabbitholegg/questdk-plugin-soundxyz Patch
@rabbitholegg/questdk-plugin-stargate Patch
@rabbitholegg/questdk-plugin-superbridge Patch
@rabbitholegg/questdk-plugin-sushi Patch
@rabbitholegg/questdk-plugin-symbiosis Patch
@rabbitholegg/questdk-plugin-synapse Patch
@rabbitholegg/questdk-plugin-tally Patch
@rabbitholegg/questdk-plugin-thruster Patch
@rabbitholegg/questdk-plugin-titles Patch
@rabbitholegg/questdk-plugin-traderjoe Patch
@rabbitholegg/questdk-plugin-treasure Patch
@rabbitholegg/questdk-plugin-uniswap Patch
@rabbitholegg/questdk-plugin-vela Patch
@rabbitholegg/questdk-plugin-woofi Patch
@rabbitholegg/questdk-plugin-zora Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@jonathandiep jonathandiep force-pushed the jon/validate-complete branch 2 times, most recently from 6a937d0 to 42c7125 Compare May 28, 2024 18:01
@jonathandiep jonathandiep changed the title Jon/validate complete feat(Boost): add complete action to Boost plugin May 28, 2024
@jonathandiep jonathandiep marked this pull request as ready for review May 28, 2024 18:16
@jonathandiep jonathandiep requested a review from a team as a code owner May 28, 2024 18:16
Copy link
Contributor

@Quazia Quazia left a comment

Choose a reason for hiding this comment

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

Looks good - let me know if you want help testing this flow with the corresponding backend changes.

@@ -40,7 +40,6 @@ export default {
},
],
plugins: [
polyfillNode(),
Copy link
Contributor

Choose a reason for hiding this comment

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

We're trying to move over to Vite for our build now (sorry for the whiplash) but why remove this for this PR?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

There was some weird build config (I think it was after importing axios into the boost plugin)

At some point, I took at look at the Neynar plugin and it didn't have the polyfillNode(). So I removed it here too and it worked

'At least one of chainId, boostId, or actionType must be provided',
},
)
export const CompleteActionDetailSchema = z.object({
Copy link
Contributor

Choose a reason for hiding this comment

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

We can rely on this export once the utils are cleaned up vs copying it over the backend. No action needed mostly a reminder for me to go back and clean this up in the RH repo once that change is live.

Copy link
Member

@jimobrien jimobrien left a comment

Choose a reason for hiding this comment

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

🔥

@jonathandiep jonathandiep force-pushed the jon/validate-complete branch from 19dc031 to 0f3a4f1 Compare May 28, 2024 19:28
@jonathandiep jonathandiep merged commit a073125 into main May 28, 2024
9 checks passed
@mmackz mmackz deleted the jon/validate-complete branch June 5, 2024 04:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants