-
Notifications
You must be signed in to change notification settings - Fork 374
fix: error message in QnA KB creation #4497
Conversation
beyackle
left a comment
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.
This PR should have the yarn l10n script run and the resulting changes to en-US.json committed alongside it.
That's a great point, Ben! Thanks for bringing it up! |
|
@beyackle FYI. |
That's the result of other earlier changes that didn't run the script. I'm okay with merging these just so the file stays up-to-date without needing another PR just to take care of those. |
Thank you! That makes sense to me. |
* Update constants.ts * run scripts and bring changes to the en-us file. Co-authored-by: Ben Yackley <[email protected]>
Description
yarn l10nscript and bring resulting changes to en-US.json committed alongside it.Before
Task Item
#4448
#minor
Screenshots
After