GitHub-ReadMe

GitHub-ReadMe copied to clipboard

Description of the bug. ✍🏻

No server-side validations on the newsletter's email field as required can be removed from the inspect and malicious actors can bypass client-side validation.

Current Behaviour

When we click on the subscribe button keeping the newsletter's email field empty it didn't give any message to enter an email or the field should not be empty.

Expected Behaviour

There should be email validations on that field which can check and identify correct email addresses at the server side.

Reproduction Steps

1. Go to a landing page
2. Click on subscribe button
3. No error message like the field is required

Screenshots

Additional information

I can fix it by using JavaScript

Contributor terms

• [X] I agree to follow the Contributing Instructions

Hello @kadambari68! Thank you for raising this issue.

Please make sure to follow our Contributing Guidelines. 💪🏻 Don't forget to ⭐ our GitHub-ReadMe. 📃

Our review team will carefully assess the issue and reach out to you soon! 😇 We appreciate your patience! 😀

I guess we need to remove this... Coz the content needs to be decided. I don't think we'll have a newsletter field. Wanna remove it?

Yes sure but will that count as a contribution too?

Yep, definitely.

ok then sure

Shall I remove all the points in the description section or is it ok to start working without it

Give your first attempt, will do more soon! 😊

Then can you assign it to me

@kadambari68 Yours...