Commit 0eecc941 authored by Alfredo Sumaran's avatar Alfredo Sumaran Committed by Alejandro Rodríguez

Merge branch '24903-allow-groups-with-special-characters' into 'master'

Allow dots in group names to pass validation for Create Group and Edit Group forms

## What does this MR do?
Allows dots within group names to pass form validation when creating or editing a group.

## Are there points in the code the reviewer needs to double check?
Nope, pretty straight forward.

## Why was this MR needed?
Although group names with dots are allowed, our frontend validation mistakenly invalidates them

## Screenshots (if relevant)

#### Before:

#### After:

## Does this MR meet the acceptance criteria?

- [ ] ~~[Changelog entry]( added~~
- [ ] ~~[Documentation created/updated](
- [ ] ~~API support added~~
- Tests
  - [ ] Added for this feature/bug
  - [ ] All builds are passing
- [x] Conform by the [merge request performance guides](
- [x] Conform by the [style guides](
- [x] Branch has no merge conflicts with `master` (if it does - rebase it please)
- [x] [Squashed related commits together](

## What are the relevant issue numbers?

Related to: #24622
Closes #24903

See merge request !7723
parent 82707c46
......@@ -13,8 +13,9 @@
= root_url
= f.text_field :path, placeholder: 'open-source', class: 'form-control',
autofocus: local_assigns[:autofocus] || false, pattern: "[a-zA-Z0-9-_]+",
required: true, title: 'Please choose a group name with no special characters.'
autofocus: local_assigns[:autofocus] || false, required: true,
pattern: Gitlab::Regex::NAMESPACE_REGEX_STR_SIMPLE,
title: 'Please choose a group name with no special characters.'
- if @group.persisted?
Markdown is supported
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment