Skip to content

Fix different error message in source and package.nls for npm package#37096

Merged
egamma merged 2 commits intomicrosoft:masterfrom
qcz:patch-4
Oct 31, 2017
Merged

Fix different error message in source and package.nls for npm package#37096
egamma merged 2 commits intomicrosoft:masterfrom
qcz:patch-4

Conversation

@qcz
Copy link
Contributor

@qcz qcz commented Oct 29, 2017

No description provided.

@egamma egamma merged commit a05e3d9 into microsoft:master Oct 31, 2017
@egamma
Copy link
Member

egamma commented Oct 31, 2017

@qcz Thanks for the fix!

@qcz qcz deleted the patch-4 branch October 31, 2017 13:49
egamma pushed a commit that referenced this pull request Oct 31, 2017
#37096)

* Fix package.nls.json for npm

Error message was different in main.ts.

* Fix language string key in npm/src/main.ts
@qcz
Copy link
Contributor Author

qcz commented Nov 6, 2017

@egamma: This PR is missing from the Thank You section of v1_18.md :)

@egamma egamma added this to the October 2017 milestone Nov 6, 2017
@egamma
Copy link
Member

egamma commented Nov 6, 2017

@qcz sorry, I forgot to assing a milestone to the PR, this is why we didn't find it. I will add it.

@qcz
Copy link
Contributor Author

qcz commented Nov 6, 2017

@egamma: No problem! Thanks!

@github-actions github-actions bot locked and limited conversation to collaborators Mar 29, 2020
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.

2 participants