[go: up one dir, main page]

Skip to content

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 to Gitlab::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 🤖