Skip to content

[PE-7237] Claim vested coins flow - #13307

Merged
dharit-tan merged 18 commits into
mainfrom
rt-vest
Oct 28, 2025
Merged

[PE-7237] Claim vested coins flow#13307
dharit-tan merged 18 commits into
mainfrom
rt-vest

Conversation

@dharit-tan

@dharit-tan dharit-tan commented Oct 23, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

Description

Migrating the claim-vested script to a relay endpoint + UI

  • New info section that shows locked/unlocked amounts
  • claim modal where user sets a percentage to go to the rewards pool (this doesn't get saved anywhere)
  • New endpoint on solana relay that returns the claim tx for the user to sign and then relay

How Has This Been Tested?

Successful tx with hardcoded amount = 2: https://explorer.solana.com/tx/4rWcfpd4tEmjfQBBSkXhPd6FA381VhqpDMjFU5GFs3NyefP2SMXYHKTcPuE7nRxxBmPxLyr4VSQe9vEEvgm99QnS

Successful tx with hardcoded amount = 1 $BELIEVE:
https://solscan.io/tx/67EKhRK9kuPd2yeBQ31yer6q6e4W4qb9e1x3gBrRZe6xANjmTbpv4gSHTPyfyhNjg9URghUFyzHDidVsxTVA8vLV#tokenBalanceChange

haven't tested with full amount yet

@changeset-bot

changeset-bot Bot commented Oct 23, 2025 •
edited
Loading

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: a7d7912

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

dharit-tan
dharit-tan commented Oct 24, 2025
View reviewed changes
Comment thread packages/web/src/hooks/useVestedCoinsInfo.ts Outdated
dylanjeffers
dylanjeffers approved these changes Oct 27, 2025
View reviewed changes

@dylanjeffers dylanjeffers left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

generally looks quite solid!

Comment thread ...ery-provider/plugins/pedalboard/apps/solana-relay/src/routes/launchpad/claim_vested_coins.ts Outdated
Comment thread ...ery-provider/plugins/pedalboard/apps/solana-relay/src/routes/launchpad/claim_vested_coins.ts Outdated
Comment thread ...ery-provider/plugins/pedalboard/apps/solana-relay/src/routes/launchpad/claim_vested_coins.ts Outdated
Comment thread ...ery-provider/plugins/pedalboard/apps/solana-relay/src/routes/launchpad/claim_vested_coins.ts
const rewardsPoolAccountInfo =
await connection.getAccountInfo(rewardsPoolAddress)
if (!rewardsPoolAccountInfo) {
throw new Error(

@dylanjeffers dylanjeffers Oct 27, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

do we want to throw for the entire script if this is the case? probably huh?

@dharit-tan dharit-tan Oct 27, 2025

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

i think so - in this branch user has specified that they want to disburse to rewards pool. if that's not possible then i think we should fail.

Comment thread ...ery-provider/plugins/pedalboard/apps/solana-relay/src/routes/launchpad/claim_vested_coins.ts
mintPublicKey, // mint
rewardsPoolAddress, // destination (rewards pool)
ownerWallet, // owner of source account
BigInt(rewardsPoolAmount.toString()), // amount as BigInt to handle large values

@dylanjeffers dylanjeffers Oct 27, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

what is rewardsPoolAmount rn? bn?

@dharit-tan dharit-tan Oct 28, 2025

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

yeah and unfortunately meteora uses BN so we're locked in

Comment thread packages/web/src/hooks/useVestedCoinsInfo.ts Outdated
Comment thread packages/web/src/hooks/useVestedCoinsInfo.ts Outdated
@socket-security

socket-security Bot commented Oct 27, 2025 •
edited
Loading

Copy link
Copy Markdown

No dependency changes detected. Learn more about Socket for GitHub.

👍 No dependency changes detected in pull request

@dharit-tan
dharit-tan merged commit 46ad234 into main Oct 28, 2025
8 of 12 checks passed
@dharit-tan
dharit-tan deleted the rt-vest branch October 28, 2025 02:46
audius-infra pushed a commit that referenced this pull request Oct 28, 2025
audius-infra
Audius Protocol v0.7.273
9232cad 
[46ad234] [PE-7237] Claim vested coins flow (#13307) Reed
[57b5340] Drop ai-attribution (#13321) Dylan Jeffers
[452326b] Revert "[PE-7204] Associate wallets on solana /relay calls" (#13314) JD Francis
audius-infra pushed a commit that referenced this pull request Nov 1, 2025
audius-infra
Audius Client (Web and Mobile) v1.5.159
f8db255 
[98da331] Claim vested coins flow QA (#13335) Reed
[0216cc9] [PE-7255] Add user balance history graph (#13337) Dylan Jeffers
[fb07b7a] [PE-7256|PE-7257] Rework artist coins list & create add cash page (#13341) JD Francis
[20bbd05] Drop donation (#13346) Dylan Jeffers
[c6de169] Fix exclusive tracks section playback (#13348) Dylan Jeffers
[eb8b147] Add error state if jup quote fails (#13344) Reed
[0903d80] Select artist-coin flair fixes (#13345) Dylan Jeffers
[e01065d] [PE-7266] Add coin exclusive tracks to coin page (#13332) Dylan Jeffers
[0aef321] Update FixedDecimal to support scientific notation (#13339) Marcus Pasell
[096aa76] [PE-7232] Fix smart tab behavior for external wallets + fix unauth buy button coins table (#13336) JD Francis
[1e2e0a6] [PE-7252] Always show user's launched coin (#13334) Reed
[da27d25] Version Packages (#13331) github-actions[bot]
[c13483e] Drop rights and covers feature flag (#13324) Dylan Jeffers
[28791a9] Add artist coin flair selector feature (#13317) Farid Salau
[2625849] [PE-7253] Fix web explore page title (#13326) Reed
[c275d11] Fix web lint (#13325) Reed
[46ad234] [PE-7237] Claim vested coins flow (#13307) Reed
[57b5340] Drop ai-attribution (#13321) Dylan Jeffers
[99e2e8c] Drop track replace flag (#13323) Dylan Jeffers
[705381c] [PE-7249] Invalidate artist-coin on buy/sell/swap (#13322) Dylan Jeffers
[87dbc94] [PE-7235] Fix pay with anything token picker (#13316) Dylan Jeffers
[bfadb5b] Revert "Artist coins page size 1000 (#13313)" (#13320) Reed
[452326b] Revert "[PE-7204] Associate wallets on solana /relay calls" (#13314) JD Francis
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@raymondjacobson raymondjacobson Awaiting requested review from raymondjacobson
@DejayJD DejayJD Awaiting requested review from DejayJD
@faridsalau faridsalau Awaiting requested review from faridsalau
@dylanjeffers dylanjeffers Awaiting requested review from dylanjeffers

Assignees

No one assigned

Labels

size/XXL

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@dharit-tan @dylanjeffers