Move the License check outside the feature flags helper
What does this MR do and why?
This MR is an adjustment to the work done in: !139064 (merged)
In this MR we:
- Move the License check outside the feature flags helper
- Move the License check to the controller
Reasons:
-
Refactored Code Suggestions Helper:
- Moved License check out of the feature flags helper.
-
Controller Integration:
- Shifted License check to the controller for better alignment.
-
Code Maintenance:
- Separated License check for future feature flag removal ease.
-
Performance Optimization:
- Avoid unnecessary License checks with each FF check outside the controller, potentially saving resources.
These changes enhance organization, maintainability, and performance.
How to set up and validate locally
Green specs should be enough.
MR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
-
I have evaluated the MR acceptance checklist for this MR.
Related to #434686
Edited by Mohamed Hamda