Skip to content

[PE-7221] Fix remove linked wallet - #13300

Merged
dylanjeffers merged 1 commit into
mainfrom
pe-7221-remove-linked-wallet
Oct 23, 2025
Merged

[PE-7221] Fix remove linked wallet#13300
dylanjeffers merged 1 commit into
mainfrom
pe-7221-remove-linked-wallet

Conversation

@dylanjeffers

@dylanjeffers dylanjeffers commented Oct 22, 2025

Copy link
Copy Markdown
Contributor

Description

Found two bugs during my investigation of remove associated wallet. Unclear if these will directly solve though.

  1. Solidly bad bug where associated wallets logic did not parametirze the and and or conditions correctly.
  2. Issue where we access json metadata before checking it's existence

I'd say lets get these through and ill continue to dig in

@changeset-bot

changeset-bot Bot commented Oct 22, 2025

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 823de66

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

@dylanjeffers
dylanjeffers merged commit ccde6ec into main Oct 23, 2025
4 of 6 checks passed
@dylanjeffers
dylanjeffers deleted the pe-7221-remove-linked-wallet branch October 23, 2025 17:05
audius-infra pushed a commit that referenced this pull request Oct 24, 2025
[292123d] Use DB to get pool addresses for fee claims (#13287) Marcus Pasell
[ccde6ec] [PE-7221] Fix remove linked wallet (#13300) Dylan Jeffers
[b9b22cd] [PE-7204] Associate wallets on solana /relay calls (#13299) JD Francis
[86198ae] [PE-7202] Add indexing support for `user.coin_flair_mint` (#13296) Randy Schott
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants