-
Notifications
You must be signed in to change notification settings - Fork 600
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: added validation to the Community Newsletter form #1977
base: master
Are you sure you want to change the base?
Conversation
Yay, your first pull request! 👍 A contributor will be by to give feedback soon. In the meantime, please review the Newcomers' Guide and sure to join the community Slack. |
Signed-off-by: aakankshabhende <[email protected]>
Signed-off-by: aakankshabhende <[email protected]>
❌ Deploy Preview for mesheryio-preview failed.
|
@aakankshabhende |
@aakankshabhende thanks for this. Are we ready for review here or still in-draft? |
Yes, its still in draft, I'll send a fix for styling and minimum length requirement and make it ready to review. |
Signed-off-by: aakankshabhende <[email protected]>
Signed-off-by: aakankshabhende <[email protected]>
Signed-off-by: aakankshabhende <[email protected]>
@Venkata-Sai-Vishwanath-robo would you like to offer review on this PR? |
Sure, @leecalcote! I would love to take it on. I’ll do my best to provide helpful feedback and appreciate your understanding as I learn through this process. |
Description
Added validation to the Community Newsletter form, the first name and last name should accept alphabetical input only.
This PR fixes #1974
Notes for Reviewers
Signed commits