Skip to content
This repository was archived by the owner on Jul 9, 2025. It is now read-only.

Conversation

@beyackle
Copy link
Contributor

@beyackle beyackle commented Sep 15, 2020

Description

There were more strings missing formatMessage calls. Also I deleted some "locales" directories inside our ui-plugins directory that aren't actually being used - all of the strings that formatMessage finds, including the ones inside that directory, get centralized out into the main en-US.json file.

Task Item

#minor - more l10n stuff

@beyackle beyackle changed the title fixes: add more missing formatMessage calls fix: add more missing formatMessage calls Sep 15, 2020
@coveralls
Copy link

coveralls commented Sep 15, 2020

Coverage Status

Coverage remained the same at 55.44% when pulling b8613e0 on beyackle/l10nFixes_uiPlugins into cf5544b on main.

@a-b-r-o-w-n a-b-r-o-w-n merged commit 5668e0b into main Sep 15, 2020
@a-b-r-o-w-n a-b-r-o-w-n deleted the beyackle/l10nFixes_uiPlugins branch September 15, 2020 23:47
alanlong9278 added a commit to alanlong9278/BotFramework-Composer that referenced this pull request Sep 16, 2020
* main:
  fix: add more missing formatMessage calls (microsoft#4144)
  feat: electron splash screen (microsoft#4119)
  feat: Add QnA files to dispatch model in skill manifest (microsoft#3985)
  fix: check whether operation is under current project folder (microsoft#4078)
  update localization and add missing "example" string (microsoft#4138)
  fix: defense invocation of `value.match()` (microsoft#4110)
  fix: checkReturnType in ExpressionValidation throws unexpected error (microsoft#4112)
  chore(deps): Bump tree-kill from 1.2.1 to 1.2.2 in /Composer (microsoft#4035)
  build: fix docker builds in ACR (microsoft#3986)
  fix: security: downgrade node-forge to 0.9.0 (microsoft#4133)
  chore: update archiver to fix security warning (microsoft#4116)
@cwhitten cwhitten mentioned this pull request Nov 13, 2020
lei9444 pushed a commit to lei9444/BotFramework-Composer-1 that referenced this pull request Jun 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants