Skip to content
This repository was archived by the owner on Aug 7, 2026. It is now read-only.

feat(WPN-1486): remove Solana Snap assets tracking after direct migration (Lane B - PR 6b) - #640

Closed
ulissesferreira wants to merge 1 commit into
WPN-1476-hardcode-core-assets-adapterfrom
WPN-1486-remove-solana-assets-tracking
Closed

feat(WPN-1486): remove Solana Snap assets tracking after direct migration (Lane B - PR 6b)#640
ulissesferreira wants to merge 1 commit into
WPN-1476-hardcode-core-assets-adapterfrom
WPN-1486-remove-solana-assets-tracking

Conversation

@ulissesferreira

Copy link
Copy Markdown
Contributor

Summary

  • Remove SnapAssetsAdapter balance fetch/persist code and AssetsRepository
  • Remove assetEntities from snap state
  • AssetsService is Core-only for all balance reads; clean up wiring in snapContext

Test plan

  • yarn workspace @metamask/solana-wallet-snap test:core (891 passed)

@ulissesferreira
ulissesferreira force-pushed the WPN-1486-remove-solana-assets-tracking branch from 0ce6db4 to 0ccec27 Compare July 31, 2026 23:06
@ulissesferreira
ulissesferreira force-pushed the WPN-1486-remove-solana-assets-tracking branch from 0ccec27 to a014762 Compare August 2, 2026 23:42
@ulissesferreira
ulissesferreira force-pushed the WPN-1486-remove-solana-assets-tracking branch 2 times, most recently from 5b99836 to b9a39d1 Compare August 3, 2026 00:02
@ulissesferreira
ulissesferreira force-pushed the WPN-1486-remove-solana-assets-tracking branch from b9a39d1 to 0edfaf4 Compare August 3, 2026 00:14
@ulissesferreira
ulissesferreira force-pushed the WPN-1486-remove-solana-assets-tracking branch from 0edfaf4 to d4d6fda Compare August 5, 2026 10:33
@sonarqubecloud

sonarqubecloud Bot commented Aug 5, 2026

Copy link
Copy Markdown

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant