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

Mapping groups #85

Closed
helgoboss opened this issue Jan 5, 2021 · 0 comments
Closed

Mapping groups #85

helgoboss opened this issue Jan 5, 2021 · 0 comments
Labels
enhancement New feature or request
Milestone

Comments

@helgoboss
Copy link
Owner

Main purpose: Allow to set activation condition for a whole group of mappings (makes many conditional activation use cases easier to manage).

Other nice usages:

  • Might make it easier to keep track if there are many mappings.
  • A whole group of mappings could be enabled/disabled.
  • Would make it much easier to implement and understand Feedback groups #14.
@helgoboss helgoboss added the enhancement New feature or request label Jan 5, 2021
@helgoboss helgoboss added this to the 1.12.0 milestone Jan 5, 2021
helgoboss added a commit that referenced this issue Jan 6, 2021
…in layer

- First activation condition will be the most significant one. It's set to
  "Always" at the moment but it will be replaced by the mapping group condition later.
- This should not have changed any behavior so far.
- It was necessary to introduce a separate activation condition to domain layer because
  the condition evaluation logic is there.
- It was also necessary to add another activation flag in order to be able to correctly
  detect the final activation result if just one of the activation conditions was affected
  by a given parameter change.
- An alternative would have been to introduce the concept of groups to the domain layer
  but this seems unnecessary as groups are just intended for easier management, which is
  responsibility of the application layer.
helgoboss added a commit that referenced this issue Jan 7, 2021
helgoboss added a commit that referenced this issue Jan 7, 2021
helgoboss added a commit that referenced this issue Jan 7, 2021
much more elegant and less code!
helgoboss added a commit that referenced this issue Jan 7, 2021
helgoboss added a commit that referenced this issue Jan 25, 2021
helgoboss added a commit that referenced this issue Jan 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant