Skip to content

Remove Rain-specific amount normalization logic - #1948

Merged
MusabShakeel576 merged 1 commit into
qafrom
claude/fix-rain-card-currency-WwtwY
Apr 15, 2026
Merged

Remove Rain-specific amount normalization logic#1948
MusabShakeel576 merged 1 commit into
qafrom
claude/fix-rain-card-currency-WwtwY

Conversation

@MusabShakeel576

Copy link
Copy Markdown
Contributor

Summary

Updated card amount normalization to remove provider-specific handling, as both Rain and Bridge now return amounts in dollars from the backend.

Key Changes

  • Simplified normalizeCardAmount() to directly parse the amount string without provider-specific conversion logic
  • Removed the conditional check that divided Rain provider amounts by 100 (cents to dollars conversion)
  • Updated JSDoc comments to reflect that both providers now return amounts in the same format (dollars)
  • Removed outdated references to "Rain amounts (cents)" from function documentation

Implementation Details

The normalizeCardAmount() function previously contained provider-specific logic to handle Rain returning amounts in cents while Bridge returned dollars. This change removes that complexity since the backend now standardizes the response format across both providers, returning amounts as dollars directly.

https://claude.ai/code/session_016jQjBTDPvPYSqZrsDmouBs

…ions

The backend now consistently returns transaction amounts in dollars for
both Rain and Bridge providers. The frontend no longer needs to divide
Rain amounts by 100, which was causing card activity to show $0.13
instead of $13.

https://claude.ai/code/session_016jQjBTDPvPYSqZrsDmouBs
@vercel

vercel Bot commented Apr 15, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

2 Skipped Deployments
Project Deployment Actions Updated (UTC)
solid-app Ignored Ignored Apr 15, 2026 8:27am
solid-app-staging Ignored Ignored Apr 15, 2026 8:27am

Request Review

@MusabShakeel576
MusabShakeel576 merged commit fcb1baa into qa Apr 15, 2026
4 of 5 checks passed
MusabShakeel576 added a commit that referenced this pull request Apr 15, 2026
…cy-WwtwY

Remove Rain-specific amount normalization logic
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.

2 participants