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

LCFS - BUG - Decimals #1462

Open
airinggov opened this issue Dec 13, 2024 · 2 comments
Open

LCFS - BUG - Decimals #1462

airinggov opened this issue Dec 13, 2024 · 2 comments
Assignees
Labels
bug Something isn't working Compliance Reporting High High priority ticket but not critical at this time

Comments

@airinggov
Copy link
Collaborator

airinggov commented Dec 13, 2024

Describe the Bug:
Fuel volume and compliance unit numbers in the Summary should not display decimal points (currency/dollar figures can have 2 decimals).

Image

Expected Behaviour:

  • Numbers (Fuel volume and compliance unit numbers) in the Summary should not display decimal points.
  • Currency figures can display decimals.

Actual Behaviour:

  • Decimal points are being shown for numbers in the Summary section where they are not required.
  • This creates an inconsistent and unnecessary level of precision for non-currency values.

Steps To Reproduce:
User/Role: Any user accessing compliance report Summary section.

@airinggov airinggov added bug Something isn't working Compliance Reporting High High priority ticket but not critical at this time labels Dec 13, 2024
@dhaselhan dhaselhan self-assigned this Dec 17, 2024
@dhaselhan
Copy link
Collaborator

Blocked pending a meeting with Al and Lindsy to verify where / when we should be rounding

@dhaselhan
Copy link
Collaborator

Decision was made to round the non money line numbers in the front end to whole integers.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Compliance Reporting High High priority ticket but not critical at this time
Projects
None yet
Development

No branches or pull requests

2 participants