Skip to content

Investigate if all classes that include HasIssueTrackerFields should validate new_issues_url

Everyone can contribute. Help move this issue forward while earning points, leveling up and collecting rewards.

The following discussion from !81541 (merged) should be addressed:

  • @.luke started a discussion: (+1 comment)

    Suggestion (not blocking)

    This is only integration that we're including HasIssueTrackerFields that has a different validates line - :new_issues_url is missing.

    We're defining new_issues_url as required in the field DSL, and this is consistent with how it was previously defined in BaseIssueTracker.

    Is this validation missing from this integration? And if so, can we move the shared validates line into HasIssueTrackerFields to keep them all consistent?

    This is not-blocking because this is a problem on master rather than something introduced by this MR, but it would make a great follow-up.

Edited by 🤖 GitLab Bot 🤖