Skip to content

Fix $AUDIO wallet history loading failure on tip transactions#14103

Closed
cursor[bot] wants to merge 1 commit intomainfrom
cursor/-bc-d45e0c38-21f5-5d51-95c6-415904535c78-01e0
Closed

Fix $AUDIO wallet history loading failure on tip transactions#14103
cursor[bot] wants to merge 1 commit intomainfrom
cursor/-bc-d45e0c38-21f5-5d51-95c6-415904535c78-01e0

Conversation

@cursor
Copy link
Copy Markdown
Contributor

@cursor cursor bot commented Apr 10, 2026

Summary

  • fix $AUDIO Transactions parsing to support tip transaction types returned by the backend
  • add TIP to shared transaction types and map SDK transactionType: 'tip' in the common adapter
  • update web transaction table/icon/details rendering to handle TIP entries cleanly
  • add adapter unit test coverage for tip mapping

Root Cause

The transaction adapter in @audius/common threw Unknown Transaction for backend tip history rows. That exception caused repeated query retries (long loading spinner) and eventual empty-state fallback in the wallet history UI.

Testing

  • Attempted: npm run test -w @audius/common -- src/adapters/audioTransactions.test.ts --run
  • Result: failed in CI environment because vitest binary is unavailable (sh: 1: vitest: not found)
Open in Web View Automation 

Co-authored-by: Ray Jacobson <raymondjacobson@users.noreply.github.com>
@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Apr 10, 2026

⚠️ No Changeset found

Latest commit: c5290f7

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

@julianbaker
Copy link
Copy Markdown
Member

Im just going to fix this with my other changes.

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