Skip to content

chore: Bump contracts & sdk for solana/kit 2.3.0 #2170

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
Open
pxrl wants to merge 15 commits into
base: master
Choose a base branch
from
Open

chore: Bump contracts & sdk for solana/kit 2.3.0 #2170

pxrl wants to merge 15 commits into from

Conversation

@pxrl
Copy link
Contributor

@pxrl pxrl commented Jan 12, 2026

No description provided.

@vercel
Copy link

vercel bot commented Jan 12, 2026 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
app-frontend-v3 Ready Ready Preview, Comment Jan 16, 2026 0:34am
sepolia-frontend-v3 Ready Ready Preview, Comment Jan 16, 2026 0:34am

Review with Vercel Agent

@vercel
Copy link

vercel bot commented Jan 12, 2026

Deployment failed with the following error:

Hobby accounts are limited to daily cron jobs. This cron expression (* * * * *) would run more than once per day. Upgrade to the Pro plan to unlock all Cron Jobs features on Vercel.

Learn More: https://vercel.link/3Fpeeb1

@pxrl pxrl marked this pull request as ready for review January 12, 2026 22:43
pxrl
pxrl commented Jan 12, 2026
View reviewed changes
e2e-api/swap/fetch-approval.test.ts
"WBNB",
"WPOL",
"WHYPE",
"TATARA-WBTC",
Copy link
Contributor Author

@pxrl pxrl Jan 12, 2026

Choose a reason for hiding this comment

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

Missed this on the recent PR to remove TATARA definitions.

test/api/_utils.test.ts
"WETH",
"WPOL",
"WHYPE",
"TATARA-WBTC",
Copy link
Contributor Author

@pxrl pxrl Jan 12, 2026

Choose a reason for hiding this comment

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

Also missed during the recent cleanup.

api/_dexes/jupiter/utils/transaction-builder.ts
Copy link
Contributor Author

@pxrl pxrl Jan 12, 2026

Choose a reason for hiding this comment

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

Claude proposed this fix to address type issues exposed by the Kit upgrade.

@pxrl pxrl mentioned this pull request Jan 13, 2026
Merged
NikolasHaimerl
NikolasHaimerl previously approved these changes Jan 13, 2026
View reviewed changes
@pxrl pxrl requested a review from NikolasHaimerl January 15, 2026 15:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mrice32 mrice32 Awaiting requested review from mrice32 mrice32 is a code owner

@nicholaspai nicholaspai Awaiting requested review from nicholaspai nicholaspai is a code owner

@dohaki dohaki Awaiting requested review from dohaki dohaki is a code owner

@gsteenkamp89 gsteenkamp89 Awaiting requested review from gsteenkamp89 gsteenkamp89 is a code owner

@amateima amateima Awaiting requested review from amateima amateima is a code owner

@0xjorgen 0xjorgen Awaiting requested review from 0xjorgen 0xjorgen is a code owner

@melisaguevara melisaguevara Awaiting requested review from melisaguevara melisaguevara is a code owner

@ashwinrava ashwinrava Awaiting requested review from ashwinrava ashwinrava is a code owner

@NikolasHaimerl NikolasHaimerl Awaiting requested review from NikolasHaimerl NikolasHaimerl is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

do not merge

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@pxrl @NikolasHaimerl