Skip to content

Move the License check outside the feature flags helper

Mohamed Hamda requested to merge 434686-mhamda-move-check-to-ui into master

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:

  1. Refactored Code Suggestions Helper:

    • Moved License check out of the feature flags helper.

  2. Controller Integration:

    • Shifted License check to the controller for better alignment.

  3. Code Maintenance:

    • Separated License check for future feature flag removal ease.

  4. 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.

Related to #434686

Edited by Mohamed Hamda

Merge request reports