Skip to content

We changed email regex patterns in one place to Angular built-in, but kept our own regex in another place. We may change both? #6006

Description

@MW3000

Describe the bug

We changed email regex patterns in one place to Angular built-in, but kept our own regex in another place. We may change both?
Previously changed email regex pattern:
#4999

Stil remaining DSpace’s own email regex pattern for the feedback form:

email: ['', [Validators.required, Validators.pattern('^[a-z0-9._%+-]+@[a-z0-9.-]+\\.[a-z]{2,4}$')]],

Expected behavior

We should validate all email address input fields identically and get rid of custom regex code.

Related work

#4999

Metadata

Metadata

Labels

Type

Projects

Status
🙋 Needs Help / Unscheduled

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions