Skip to content

Only log error on first occurrence of missing substitution#9096

Merged
Gudahtt merged 1 commit intodevelopfrom
only-log-error-for-first-occurrence-of-missing-substitutions
Jul 29, 2020
Merged

Only log error on first occurrence of missing substitution#9096
Gudahtt merged 1 commit intodevelopfrom
only-log-error-for-first-occurrence-of-missing-substitutions

Conversation

@Gudahtt
Copy link
Copy Markdown
Member

@Gudahtt Gudahtt commented Jul 28, 2020

A missing substitution for a localized message will now only log an error upon the first occurrence. Further errors are generally not useful.

A missing substitution for a localized message will now only log an
error upon the first occurrence. Further errors are generally not
useful.
@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [af01d28]
Page Load Metrics (668 ± 54 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint319048188
domContentLoaded34580466611354
load34780566811354
domInteractive34580466611354

@Gudahtt Gudahtt marked this pull request as ready for review July 28, 2020 19:59
@Gudahtt Gudahtt requested a review from a team as a code owner July 28, 2020 19:59
Copy link
Copy Markdown
Contributor

@whymarrh whymarrh left a comment

Choose a reason for hiding this comment

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

LGTM!

@Gudahtt Gudahtt merged commit b7715f6 into develop Jul 29, 2020
@Gudahtt Gudahtt deleted the only-log-error-for-first-occurrence-of-missing-substitutions branch July 29, 2020 16:09
Gudahtt added a commit that referenced this pull request Jul 30, 2020
* origin/develop: (582 commits)
  Use async/await for seedPhraseVerifier.verifyAccounts (#9100)
  Use async/await for getRestrictedMethods (#9099)
  Update dependencies (#9105)
  update email us to contact us (#9104)
  Improve source maps (#9101)
  Update font family globally (#9073)
  rpc-cap@3.1.0 (#9103)
  Use environment variable for production Sentry DSN (#9097)
  Only log error on first occurrence of missing substitution (#9096)
  Use mixins for typography instead of placeholder selectors (#9072)
  Update css folder structure (#9071)
  Disable Sentry in development (#9095)
  Use environment variable for MetaMetrics project ID (#9094)
  Use development metametrics project during tests (#9093)
  json-rpc-engine@5.2.0 (#9091)
  fixup! call initializeProvider where necessary
  call initializeProvider where necessary
  Add euclid fontface (#9018)
  fix timing-reliant network controller test
  Robustify permissions controller requestUserApproval tests (#9064)
  ...
Gudahtt added a commit that referenced this pull request Aug 7, 2020
A missing substitution for a localized message will now only log an
error upon the first occurrence. Further errors are generally not
useful.
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.

3 participants