Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Fix add / remove locale from configuration #1325

Merged
merged 2 commits into from
Sep 19, 2021
Merged

fix: Fix add / remove locale from configuration #1325

merged 2 commits into from
Sep 19, 2021

Conversation

Jason3S
Copy link
Collaborator

@Jason3S Jason3S commented Sep 19, 2021

Logic

Scope

When adding / removing locales, the scope is either Global or Workspace.
Global - means it is possible to adjust any configuration.
Workspaces - means it is possible to adjust any configuration that is not Global.

  • Targets are presented from Locale to Global.
  • Targets Inherit their values from Global to Local.

Adding Locale

  1. Only present targets that will have an impact.
  2. If no targets will have an impact, then present any target that can be changed.
  3. If there is only 1 target, then perform action without asking.

Removing Locale

  1. Only present targets that will have an impact.
  2. If no targets will have an impact, do nothing.
  3. If there is only 1 target, then perform action without asking.

## Logic

### Scope

When adding / removing locales, the scope is either Global or Workspace.
Global - means it is possible to adjust any configuration.
Workspaces - means it is possible to adjust any configuration that is not Global.

-   Targets are presented from Locale to Global.
-   Targets _Inherit_ their values from Global to Local.

### Adding Locale

1. Only present targets that will have an impact.
1. If no targets will have an impact, then present any target that can be changed.
1. If there is only 1 target, then perform action without asking.

### Removing Locale

1. Only present targets that will have an impact.
1. If no targets will have an impact, do nothing.
1. If there is only 1 target, then perform action without asking.
@github-actions github-actions bot added the fix PR Fix label Sep 19, 2021
@Jason3S Jason3S merged commit b5699ef into main Sep 19, 2021
@Jason3S Jason3S deleted the dev-config branch September 19, 2021 13:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix PR Fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant