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

Display "--" for percentage number when divisor is 0 #35569

Merged
merged 1 commit into from
Dec 31, 2024

Conversation

jingcheng16
Copy link
Contributor

@jingcheng16 jingcheng16 commented Dec 31, 2024

Product Description

image

Technical Summary

It's been captured by QA's post-deploy verification on production. Related comment.
We already return "--" when format percentage number. But the javascript code only takes care of number or the string of number. Add a new condition to return "--" when receive "--"

Feature Flag

No

Safety Assurance

Safety story

Straightforward frontend change

Automated test coverage

No

QA Plan

No

Rollback instructions

  • This PR can be reverted after deploy with no further considerations

Labels & Review

  • Risk label is set correctly
  • The set of people pinged as reviewers is appropriate for the level of risk of the change

@jingcheng16 jingcheng16 requested a review from mjriley December 31, 2024 15:34
@jingcheng16 jingcheng16 marked this pull request as ready for review December 31, 2024 15:34
@jingcheng16 jingcheng16 requested a review from nospame December 31, 2024 15:34
@jingcheng16 jingcheng16 merged commit 747e096 into master Dec 31, 2024
12 checks passed
@jingcheng16 jingcheng16 deleted the jc/fix-total-display-in-enterprise-tile branch December 31, 2024 17:24
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