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

fix: Group MicrosoftAppId and MicrosoftPassword together in the settings page#3181

Merged
cwhitten merged 3 commits intomasterfrom
wenyluo/group
May 26, 2020
Merged

fix: Group MicrosoftAppId and MicrosoftPassword together in the settings page#3181
cwhitten merged 3 commits intomasterfrom
wenyluo/group

Conversation

@VanyLaw
Copy link
Contributor

@VanyLaw VanyLaw commented May 26, 2020

Description

Set sensitive field into '' instead of removing those fields when filtering settings before saving. It can keep the settings structure as initialized generated and not enforce a particular structure.

Task Item

close #3046

Screenshots

@cwhitten cwhitten merged commit 3da7009 into master May 26, 2020
@cwhitten cwhitten deleted the wenyluo/group branch May 26, 2020 23:49
lei9444 pushed a commit to lei9444/BotFramework-Composer-1 that referenced this pull request Jun 15, 2021
Co-authored-by: Chris Whitten <christopher.whitten@microsoft.com>
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.

Group MicrosoftAppId and MicrosoftPassword together in the settings page

2 participants