Skip to content
Snippets Groups Projects

Unify Security Configuration between EE and CE

Merged Tetiana Chupryna requested to merge 339023-unify-configuration-ee-ce into master
All threads resolved!

What does this MR do and why?

The Security Configuration page needs to show different information depending on the tier the given project is in. Currently, the logic of that behavior is scattered in various places on the frontend and backend, which makes adding features or moving features to different tiers difficult and error-prone. read more in the issue description

In this MR we add a response for the CE version of SecurityConfigurationController for json request, so it returns configuration data. This change is behind the feature flag. It's needed until frontend is adjusted as well

Screenshots or screen recordings

These are strongly recommended to assist reviewers and reduce the time to merge your change.

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

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 #339023 (closed)

Edited by Tetiana Chupryna

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • added 1 commit

    Compare with previous version

  • Etienne Baqué approved this merge request

    approved this merge request

  • Etienne Baqué resolved all threads

    resolved all threads

  • Etienne Baqué enabled an automatic merge when the pipeline for 3f6a20b3 succeeds

    enabled an automatic merge when the pipeline for 3f6a20b3 succeeds

  • Etienne Baqué mentioned in commit 356491bd

    mentioned in commit 356491bd

  • added workflowcanary label and removed workflowstaging label

  • added workflowproduction label and removed workflowcanary label

  • Zamir Martins mentioned in merge request !108854 (closed)

    mentioned in merge request !108854 (closed)

  • Please register or sign in to reply
    Loading