Show required field error message if the field is required #318

girishpanchal30 · 2024-07-03T11:27:30Z

I've added an if condition to exclude the required field from the validation process.

Check before Pull Request is ready:

I have written a test and included it in this PR
I have run all tests and they pass
The code passes when running the PHP CodeSniffer
Code meets WordPress Coding Standards for PHP, HTML, CSS and JS
Security and Sanitization requirements have been followed
I have assigned a reviewer or two to review this PR (if you're not sure who to assign, we can do this step for you)

Closes #316

pirate-bot · 2024-07-03T11:28:28Z

Plugin build for fce71ac is ready 🛎️!

pirate-bot · 2024-07-03T14:38:19Z

🎉 This PR is included in version 32.0.24 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

girishpanchal30 added 2 commits July 3, 2024 16:52

fix: show required field error message if the field is required

0b68925

Improve if condition

fce71ac

girishpanchal30 added the pr-checklist-skip Allow this Pull Request to skip checklist. label Jul 3, 2024

girishpanchal30 requested a review from preda-bogdan July 3, 2024 11:27

girishpanchal30 linked an issue Jul 3, 2024 that may be closed by this pull request

Fields appear as required even if they're not #316

Closed

pirate-bot added the pr-checklist-complete The Pull Request checklist is complete. (automatic label) label Jul 3, 2024

girishpanchal30 mentioned this pull request Jul 3, 2024

Fields appear as required even if they're not #316

Closed

preda-bogdan approved these changes Jul 3, 2024

View reviewed changes

vytisbulkevicius merged commit 666663d into development Jul 3, 2024
6 checks passed

pirate-bot added the released Indicate that an issue has been resolved and released in a particular version of the product. label Jul 3, 2024