Skip to content

Conversation

@amir-microsoft
Copy link
Contributor

Removing locale validation:
Current locale validation doesn't support all cases, such as: de, zh-Hant-TW, En-au,aZ_cYrl-aZ
The locale is validated in the WebChat app anyway so it's redundant.

Current locale validation doesn't support all cases, such as: de, zh-Hant-TW, En-au,aZ_cYrl-aZ
The locale is validated in the WebChat app anyway so it's redundant.
@barneaoren barneaoren self-requested a review April 7, 2020 16:59
@amir-microsoft amir-microsoft merged commit f97e69e into master Apr 7, 2020
@amir-microsoft amir-microsoft deleted the bugfix/amirt/locale_regex branch April 8, 2020 12:20
amir-microsoft added a commit that referenced this pull request Jun 15, 2020
* Supporting Windows and Linux web apps. Linux by default (#57)

* Supporting Windows and Linux web apps. Linux by default

* bug fix

* update readme

* adding doc

* update readme

* Origin/deployment script update (#64)

* Removing locale validation: (#63)

Current locale validation doesn't support all cases, such as: de, zh-Hant-TW, En-au,aZ_cYrl-aZ
The locale is validated in the WebChat app anyway so it's redundant.

* Dynamic branch and repo for deploy (#70)

* When using "deploy to azure" button, taking the repo and branch from the page where the button was clicked

* Adding example for using Content security policy to limit which domains are allowed to embed this page as a frame

* remove private preview from readme (#76)

* remove private preview from readme

* Description change

Co-authored-by: Guy Becker <gubecker@microsoft.com>

* Update index.html

* Fixing a bug where a redundant parameters was passed to the function passed to

Co-authored-by: Elad Iwanir <13205761+eladiw@users.noreply.github.com>
Co-authored-by: GuyBeckerMicrosoft <61974899+GuyBeckerMicrosoft@users.noreply.github.com>
Co-authored-by: Guy Becker <gubecker@microsoft.com>
Co-authored-by: AdamWalkerMicrosoft <40535367+AdamWalkerMicrosoft@users.noreply.github.com>
Co-authored-by: amir-microsoft <>
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