Skip to content

[PE-7263] Fix remove associated wallet - #13333

Closed
dharit-tan wants to merge 2 commits into
mainfrom
rt-wallet
Closed

[PE-7263] Fix remove associated wallet#13333
dharit-tan wants to merge 2 commits into
mainfrom
rt-wallet

Conversation

@dharit-tan

Copy link
Copy Markdown
Contributor

Description

Bug: removing associated wallet was broken on stage (dharit3) and prod (dharit) for my users.

I discovered that there was another row with the same wallet address with is_delete set to true.
Screenshot 2025-10-29 at 2 13 23 PM

Not sure where that came from, but this fixes.

@rickyrombo would this have anything to do with the prod $AUDIO on stage change?

How Has This Been Tested?

Tested by modifying code on stage DN1 and removing wallet from stage client.

@changeset-bot

changeset-bot Bot commented Oct 29, 2025

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: d969b56

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

.filter(
AssociatedWallet.wallet.in_(entities_to_fetch["AssociatedWallet"]),
AssociatedWallet.is_current == True,
AssociatedWallet.is_delete == False,

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.

does this actually work? we tried something similar when doing comments changes and caused indexing stalls #13329

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.

it worked for me by testing live on stage DN1

@rickyrombo

Copy link
Copy Markdown
Contributor

@rickyrombo would this have anything to do with the prod $AUDIO on stage change?

No I don't think so

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