While I may not have made large contributions, I have provided changes in the past to provide standardization in message output format (such as for AddNetwork, DelNetwork, AddUser, DelUser, etc.) to both increase verbosity but to also provide information to the end user in a readable form (https://github.com/znc/znc/pull/353, https://github.com/znc/znc/pull/295). Most recently, my last revision was extremely minor but enforces standardization of commands to those forms (https://github.com/znc/znc/pull/933). I also (band-aid) fixed a display issue on webadmin for the SSLTrustedFingerprints caption (https://github.com/znc/znc/pull/859).
To that end, and the fact that the standardization and verbosity I set up is still in use today in a good portion of modules, I'd like my name added to the contributors/helpers list, please. (I understand if I am not added, however I would like to be included on the list)
The User Deleted output for a success does not match the rest of the output formats for other commands and needs to match the same 'standard' for outputs.
Originally at the time of editing, the "Installation is done with the ..." command blurb comes *before* the notice about if you're building from git. I believe it is more sane to have that come first in the README section for installing, as it will be seen before the actual installation commands are, so people will first see that if they are building from git they'll need to run `./autogen.sh`.
The ClearBindHost and ClearUserBindHost success messages do not make a distinction on whether it's a network bind host that's being cleared or whether it's a user bind host that's being cleared. I think that this should be reworded to make the distinction.
This modifies line 897 to have more verbosity on the output for a successful execution of the "Disconnect" command, which will allow for the output to say what network was force-disconnected on what user.
This is a repaired version of pull request #352 which I closed after a FTBFS which I did not realize.
Thomas Ward