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
[NAE-1660] Filter action API #83
Conversation
- add action delegate methods to manipulate filters, preference items
- extend methods in ActionDelegate - update processes org_group and preference_filter_item with new transitions to manipulate their data - add new method to NextGroupService - add test process
- fix set events (add ids) - maintain case consistency in changeFilter, changeMenuItem - add some todos
- cosmetic changes - fix incorrect method logic - update roles in org_group.xml
- update process (preference_filter_item) - add all roles to system - search using elastic in filter api methods
- write tests - change NextGroupService method (elastic search instead of mongo) - fix javadocs
- remove uri from filter methods for simplification - add test
- improve test
- remove demo runner code - fix query
-fix test
- fix test
- refactor mongo searching in org_group for better performance
This PR has Quantification details
Why proper sizing of changes matters
Optimal pull request sizes drive a better predictable PR flow as they strike a
What can I do to optimize my changes
How to interpret the change counts in git diff output
Was this comment helpful? 👍 :ok_hand: :thumbsdown: (Email) |
# Conflicts: # src/main/groovy/com/netgrif/application/engine/startup/DefaultFiltersRunner.groovy # src/main/groovy/com/netgrif/application/engine/startup/DemoRunner.groovy
🎉 All dependencies have been resolved ! |
- fix code smells
SonarCloud Quality Gate failed. |
Description
ActionDelegate Filter API, with changes to preexisting engine processes such as org_group, preference_filter_item
Implements NAE-1660
Dependencies
Third party dependencies
Blocking Pull requests
How Has Been This Tested?
FilterApiTest class and manual testing with a helper process
Test Configuration
Checklist: