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
Validate Group Fields in API #1121
Validate Group Fields in API #1121
Conversation
Codecov ReportBase: 92.09% // Head: 92.09% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## master #1121 +/- ##
==========================================
- Coverage 92.09% 92.09% -0.01%
==========================================
Files 172 172
Lines 6945 6954 +9
Branches 158 161 +3
==========================================
+ Hits 6396 6404 +8
- Misses 549 550 +1
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor tweak
Fixes issue #1120.
Changes in this pull request:
name
and/oremail
fields empty. Now we cannot create a group with emptyname
and/oremail
fields through API and we'll get the messageGroup name and email cannot be empty.
when either or both is empty.