[x] Tests for the changes have been added (for bug fixes / features) (if applicable)
[x] Header has been added to all new source files (run build/UpdateHeaders.bat)
[ ] Contains NO breaking changes
Breaking changes
The login panel and the management panel are two different usage scenarios, and their HeaderText may not be mixed, so I made some modifications:
Modify the original HeaderText to AddAccountHeaderText.
Add a new property ManageAccountHeaderText is used to display in the account management panel.
The constructor parameters of AccountsSettingsPaneConfig have been adjusted.
Other information
A new object has been added: WebAccountCommandParameter. It is not used as an IList property in the definition of AccountsSettingsPaneConfig, because currently WindowsProvider only supports single account.
Thanks Richasy for opening a Pull Request! The reviewers will test the PR and highlight if there is any merge conflict or changes required. If the PR is approved we will proceed to merge the pull request 🙌
PR Type
What kind of change does this PR introduce?
Feature
Support display account management panel:
What is the current behavior?
At this stage,
WindowsProvider
provides login and logout functions, the logout process does not require user involvement.What is the new behavior?
Some usage scenarios may need to display the currently logged-in user and provide some custom operations.
PR Checklist
Please check if your PR fulfills the following requirements:
Breaking changes
The login panel and the management panel are two different usage scenarios, and their
HeaderText
may not be mixed, so I made some modifications:HeaderText
toAddAccountHeaderText
.ManageAccountHeaderText
is used to display in the account management panel.AccountsSettingsPaneConfig
have been adjusted.Other information
A new object has been added:
WebAccountCommandParameter
. It is not used as anIList
property in the definition ofAccountsSettingsPaneConfig
, because currentlyWindowsProvider
only supports single account.