search

Icinga / icingaweb2-module-director

The Director aims to be your new favourite Icinga config deployment tool. Director is designed for those who want to automate their configuration deployment and those who want to grant their “point & click” users easy access to the configuration.
https://icinga.com/docs/director/latest
GNU General Public License v2.0
413 stars 203 forks source link

Fix typo in SNMP Data Fields example documentation #2931

Closed sam-banks closed 2 weeks ago

sam-banks commented 3 weeks ago

Hi,

I was reading the Icinga Director documentation and spotted a typo in the Data Fields example: SNMP page. I've included a patch to fix the typo.

Thanks!

cla-bot[bot] commented 3 weeks ago

Thank you for your pull request. Before we can look at it, you'll need to sign a Contributor License Agreement (CLA).

Please follow instructions at https://icinga.com/company/contributor-agreement to sign the CLA.

After that, please reply here with a comment and we'll verify.

Contributors that have not signed yet: @sam-banks

- If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Please [contact us](https://icinga.com/company/contact/) if you think this is the case. - If you signed the CLA as a corporation, your GitHub username may not have been submitted to us. Please reach out to the responsible person in your organization.
sam-banks commented 3 weeks ago

I've signed and submitted the CLA, can you please verify?

lippserd commented 3 weeks ago

@sam-banks Thank you for taking the time to submit the PR.

@cla-bot check

sam-banks commented 2 weeks ago

Thanks for accepting the changes @lippserd. Is there anything further required on my end? Should this issue remain open?

lippserd commented 2 weeks ago

Thanks for accepting the changes @lippserd. Is there anything further required on my end? Should this issue remain open?

Nope, I will just merge it.

sam-banks commented 2 weeks ago

Thanks again :)