Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(EMS-3546): no PDF - Export contract - Agent charges - Fixed sum commas #2678

Merged
merged 1 commit into from
Jul 4, 2024

Conversation

Zainzzkk
Copy link
Collaborator

@Zainzzkk Zainzzkk commented Jul 4, 2024

Introduction ✏️

This PR fixes an issue where fixed sum went to problem with service when submitting a number with commas

Resolution ✔️

  • Added strip commas to mapping
  • Added tests

@Zainzzkk Zainzzkk added the fix Fixes an existing issue label Jul 4, 2024
@Zainzzkk Zainzzkk self-assigned this Jul 4, 2024
@ttbarnes ttbarnes merged commit 26c1b6e into main-application-no-pdf Jul 4, 2024
167 of 168 checks passed
@ttbarnes ttbarnes deleted the fix/EMS-3546 branch July 4, 2024 15:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix Fixes an existing issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants