-
Notifications
You must be signed in to change notification settings - Fork 287
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
Allow user_medias to be optional. #264
Conversation
Codecov Report
@@ Coverage Diff @@
## main #264 +/- ##
==========================================
+ Coverage 78.40% 78.48% +0.08%
==========================================
Files 19 19
Lines 2663 2673 +10
Branches 676 684 +8
==========================================
+ Hits 2088 2098 +10
Misses 376 376
Partials 199 199
Continue to review 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.
Yes, something like this :)
Thank you for this patch! https://docs.ansible.com/ansible/latest/community/development_process.html#changelogs |
Co-authored-by: sky-joker <[email protected]>
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.
LGTM
Thank you @elrondvega :)
SUMMARY
Allows user_medias to be optional when not set.
fixes #261
ISSUE TYPE
COMPONENT NAME
zabbix_user
ADDITIONAL INFORMATION
Sets the default to None and handles when value is None.