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.

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

fix: FORMS-1560 updated validation message for revising status recipient emails section #1581

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

Conversation

revanth-banala
Copy link
Collaborator

Description

When a user try to search another recipient email rather than default one and if it does not exist, wrong validation message shows up. Needs to modify the message to “No results found, Add team members on draft/submission manage page” instead of current message “No Form Reviewers found with the search, Add form reviewers on the manage page"
Acceptance Criteria

User can search recipient emails on recipient email section and if its not found, they should able to know where to add team members so that it should show up on search

Type of Change

fix (a bug fix)

Checklist

  • I have read the CONTRIBUTING doc
  • I have checked that unit tests pass locally with my changes
  • I have run the npm script lint on the frontend and backend
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)
  • I have approval from the product owner for the contribution in this pull request

Further comments

@revanth-banala revanth-banala marked this pull request as ready for review January 17, 2025 22:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant