Follow-up from "Fix project import error occurring due to default visibility"
Everyone can contribute. Help move this issue forward while earning points, leveling up and collecting rewards.
The following discussion from !53827 (merged) should be addressed:
-
discussion: (+6 comments) @jameslopez: Should this be fixed via
Gitlab::VisibilityLevelChecker
instead?🤔 @georgekoltsov: Ideally this new validation could be a great opportunity to extend
Gitlab::VisibilityLevelChecker
but it might be a bigger task, since we would probably need to also add a reason for restriction toGitlab::VisibilityEvaluationResult
and if a reason is 'not_allowed_by_group' - set project visibility level to group visibility level. This could be done in a follow up in order to not block progress of this MR. So this change looks good to me👍
Edited by 🤖 GitLab Bot 🤖