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

Conversation

@GeoffCoxMSFT
Copy link
Member

Description

The 17 supported languages need to be in the dropdown.

Task Item

Fixed #4737

Screenshots

image

@coveralls
Copy link

coveralls commented Nov 10, 2020

Coverage Status

Coverage remained the same at 56.292% when pulling 396e435 on gcox/enable-languages into 0ab4c79 on main.

Copy link
Contributor

@beyackle beyackle left a comment

Choose a reason for hiding this comment

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

Looks good, just one tiny change needed.

const renderElectronSettings = isElectron();

const languageOptions = [{ key: 'en-US', text: formatMessage('English (US)') }];
const languageOptions = [
Copy link
Contributor

Choose a reason for hiding this comment

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

This can be moved to the outside of the component code.

beyackle
beyackle previously approved these changes Nov 10, 2020
@cwhitten cwhitten merged commit 8df175a into main Nov 10, 2020
@cwhitten cwhitten deleted the gcox/enable-languages branch November 10, 2020 22:57
@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
* Added languages to dropdown in settings

* Fixed typo in zh-Hant key

* Moved englishName to comments

* log over error to allow tests to run
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.

Localized languages should appear in dropdown in settings

7 participants