Skip to content

fix(web): parse pool total shares#385

Open
pollychen-lab wants to merge 2 commits into
TrusTrove:mainfrom
pollychen-lab:fix/issue-265-pool-total-shares
Open

fix(web): parse pool total shares#385
pollychen-lab wants to merge 2 commits into
TrusTrove:mainfrom
pollychen-lab:fix/issue-265-pool-total-shares

Conversation

@pollychen-lab

Copy link
Copy Markdown

Closes #265.

Summary

  • Populate totalShares from raw.total_shares in the web pool stats transformer.
  • Add focused regression coverage for the transformer parser.
  • Extend the existing API parser test to assert totalShares.

Validation

  • corepack pnpm --config.dangerouslyAllowAllBuilds=true --dir apps/web exec vitest run lib/transformers.test.ts
  • corepack pnpm --config.dangerouslyAllowAllBuilds=true --dir apps/web exec vitest run lib/api.test.ts -t parseRawPoolStats

Note: corepack pnpm --config.dangerouslyAllowAllBuilds=true --filter @trusttrove/sdk build && corepack pnpm --config.dangerouslyAllowAllBuilds=true --filter web exec tsc --noEmit still fails on existing unrelated web type errors, including missing useTokenAllowance/initApiClientWithToken symbols and stale component/test types.

@vercel

vercel Bot commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

@pollychen-lab is attempting to deploy a commit to the K1NGD4VID Team on Vercel.

A member of the Team first needs to authorize it.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Easy] fix(web): parseRawPoolStats must populate totalShares

1 participant