Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix email validation in contacts screen #2818

Open
wants to merge 1 commit into
base: bleeding
Choose a base branch
from
Open

Fix email validation in contacts screen #2818

wants to merge 1 commit into from

Conversation

danigargar
Copy link
Contributor

Fixed validation of non required email field when it is an empty string

Type Of Change

  • Small bug fix
  • New feature or enhancement
  • Breaking change

Checklist:

  • Commits are named and have tag following commit rules
  • Commits are split per component (schema, portal/platform, kamusers, agis, ..)
  • Changes have been tested locally
  • Fixes an existing issue (Fixes #XXXX)
  • Upport from existing Pull request #XXXX

Description

Additional information

@danigargar danigargar added +bug Bug core Core Libraries *tempest* IvozProvider 4.x Tempest Release labels Nov 15, 2024
@danigargar danigargar added this to the 4.3.0 milestone Nov 15, 2024
@danigargar danigargar self-assigned this Nov 15, 2024
ironArt3mis
ironArt3mis requested changes Nov 15, 2024
View reviewed changes
Copy link
Contributor

@ironArt3mis ironArt3mis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Functional review required

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ironArt3mis ironArt3mis ironArt3mis requested changes

@rbatistadev rbatistadev rbatistadev approved these changes

Requested changes must be addressed to merge this pull request.

Assignees

@danigargar danigargar

Labels
+bug Bug core Core Libraries functional-review *tempest* IvozProvider 4.x Tempest Release
Milestone
4.3.0
Development

Successfully merging this pull request may close these issues.
3 participants
@danigargar @ironArt3mis @rbatistadev