Skip to content

TSA-429: [FE] Allow no more than two decimal places in the Raise the limit form - #498

Open
IgnacioRB wants to merge 7 commits into
mainfrom
feat/TSA-429-no-more-than-two-decimals-raise-limit-form
Open

IgnacioRB wants to merge 7 commits into
mainfrom
feat/TSA-429-no-more-than-two-decimals-raise-limit-form

Conversation

@IgnacioRB

Copy link
Copy Markdown
Collaborator

Closes #429

Summary

This PR restricts the input in the "Raise the limit" form to a maximum of two decimal places, aligning the allowed input format with the displayed money format.

Changes

  1. Decimal place restriction in the New limit field:
    • Before: The form displayed amounts with two decimal places (e.g., $0.18 spent) but allowed users to enter values with more decimal places until four, leading to inconsistent behaviors where certain four-digit values were accepted while others were rejected without clear criteria.
    • Now: Restricts the "New limit" input to accept at most two digits after the decimal point (no decimals, one, or two decimals), matching the currency format shown to the user and displays an inline validation message immediately when the user enters three or more decimal places, preventing the request from being sent. Additionally, it automatically clears the validation error message as soon as the user corrects the value, while continuing to store and calculate actual model costs at their existing precision.
BudgetNoMoreThanTwoDecimals

@IgnacioRB IgnacioRB self-assigned this Sep 28, 2026
@IgnacioRB IgnacioRB added enhancement New feature or request frontend Frontend application labels Sep 28, 2026
@IgnacioRB IgnacioRB added this to the transcripta-release-3 milestone Sep 28, 2026

@yurii-tymoshevskyi yurii-tymoshevskyi left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fix conflicts

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

Labels

enhancement New feature or request frontend Frontend application

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

[FE] Allow no more than two decimal places in the Raise the limit form

2 participants