Skip to content

Hide user attributes behind a feature flag

Marcel van Remmerden requested to merge remove-user-attributes into master

What does this MR do and why?

There is unclarity in !95796 (merged) about whether removing all user attributes from both system notes and discussions is a good idea, so we could first test this behind a feature flag. That feature flag will only be enabled for gitlab-org and gitlab-com. If we don't find any new negative feedback that's not already covered in #372234 (closed), we will then merge !95796 (merged), which completely removes these attributes and adapts all tests.

Screenshots or screen recordings

Before After
image image

How to set up and validate locally

bin/rails c
Feature.enable(:remove_user_attributes)

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Marcel van Remmerden

Merge request reports

Loading