-
Notifications
You must be signed in to change notification settings - Fork 13.2k
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: Typo in the positionJSON
too large warning
#13467
Conversation
* Fix typo in the warning that gets displayed with the `positionJSON` is too large/long.
positionJSON
is too large/long.positionJSON
too large warning
Codecov Report
@@ Coverage Diff @@
## master #13467 +/- ##
=======================================
Coverage 77.62% 77.62%
=======================================
Files 962 962
Lines 49017 49017
Branches 6155 6155
=======================================
Hits 38050 38050
Misses 10763 10763
Partials 204 204
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
@mrshu Can you resolve the conflicts?
Thanks @michael-s-molina -- I believe the PR should be ready to be merged. |
* Fix typo in the warning that gets displayed with the `positionJSON` is too large/long.
* Fix typo in the warning that gets displayed with the `positionJSON` is too large/long.
* Fix typo in the warning that gets displayed with the `positionJSON` is too large/long.
SUMMARY
positionJSON
is too large/long.TEST PLAN
There is a good chance the localization depends on this string's exact value. I am not sure how to resolve that but I'd happily do that, should anyone provide some pointers.
ADDITIONAL INFORMATION