Skip to content

Fix: Remove blank amount from transaction entry parameters#1178

Merged
jjmata merged 1 commit into
mainfrom
seer/fix/remove-blank-amount
Mar 11, 2026
Merged

Fix: Remove blank amount from transaction entry parameters#1178
jjmata merged 1 commit into
mainfrom
seer/fix/remove-blank-amount

Conversation

@sentry

@sentry sentry Bot commented Mar 10, 2026

Copy link
Copy Markdown
Contributor

Fixes SURE-APP-N2. The issue was that: Empty amount string type-casts to nil, causing amount_money to return nil, leading to unary negation on nil in the view.

  • Deletes the ':amount' key from 'entry_params' if its value is blank (e.g., nil or empty string).
  • Ensures consistent handling of blank amounts before further processing and type conversion.
  • Prevents potential errors or unexpected behavior when the amount is not provided.

This fix was generated by Seer in Sentry, triggered automatically. 👁️ Run ID: 911453

Not quite right? Click here to continue debugging with Seer.

jjmata pushed a commit that referenced this pull request Mar 11, 2026
Approve with minor suggestions — clean fix for NoMethodError on nil amount_money.

https://claude.ai/code/session_01TvS2UQEfAQYtZMefncNsXc
@jjmata jjmata self-assigned this Mar 11, 2026
@jjmata jjmata self-requested a review March 11, 2026 16:45
@jjmata jjmata added this to the v0.6.9 milestone Mar 11, 2026
@jjmata jjmata merged commit d7a3b0c into main Mar 11, 2026
6 checks passed
@jjmata jjmata deleted the seer/fix/remove-blank-amount branch March 21, 2026 10:52
binarygituser referenced this pull request in binarygituser/sure-dev Apr 26, 2026
Co-authored-by: sentry[bot] <39604003+sentry[bot]@users.noreply.github.com>
jaxver pushed a commit to jaxver/sure that referenced this pull request May 31, 2026
…e#1178)

Co-authored-by: sentry[bot] <39604003+sentry[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

No open projects

Development

Successfully merging this pull request may close these issues.

1 participant