search

zabbix-tooling / zabbix-ldap-sync

Sync Zabbix with LDAP directory server
BSD 3-Clause "New" or "Revised" License
49 stars 35 forks source link

update_media update fixes #8 #17 #18

Closed ofbeaton closed 4 years ago

ofbeaton commented 4 years ago

fix update_media so that it doesn't erase the user's current media types, only updates/adds the one you want, side effect is it won't delete orphan media types anymore (thanks @towster)

ofbeaton commented 4 years ago

note I did not include the change to get_mediatype_id that @towster proposed in #17, as that caused problems in my install.

scoopex commented 4 years ago

Seems to be ok and to work here :-) I merge it :-)

ofbeaton commented 4 years ago

Thanks for merging!

ofbeaton commented 4 years ago

Oh wait, I messed up the PR didn't I. I think it got merged into the update_media branch instead of the master branch. Not what I meant to submit at all. Sorry I'm not used to making PRs at all. Want me to try to resubmit against master?