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

Chetan hotfix email validation #1133

Merged
merged 2 commits into from
Oct 20, 2024

Conversation

Chetan0606
Copy link
Contributor

Description

Hotfix for PR 2559

Only error messages are displayed and successful email messages are not displayed

Related PRS (if any):

To test this backend PR you need to checkout the development branch of frontend PR.

Main changes explained:

  • Import was not added after making the changes in the backend which now caused issue in frontend

How to test:

  1. check into current branch
  2. do npm install and npm run build and npm start to run this PR locally
  3. Clear site data/cache
  4. log as owner/admin user
  5. go to dashboard→ Other Links→ Send Emails → Enter the email list separated by comma → Click send email to specific user
  6. verify if you are getting an invalid email message in the top right corner if you have entered an invalid email and a success message when it's a valid one.

Screenshots or videos of changes:

Recording.2024-10-12.231553.mp4

@one-community one-community merged commit fd7d56e into development Oct 20, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants