Addition of comment based help

[Celerium]

Addresses issue #56

Quite a bit to go over with this initial update so ZERO rush to get this in place but overall 99% of the changes are the addition of comment-based help to all wrapper functions with support for the Get-Help <command name> -online function.

Example: Get-Help Get-ITGlueUsers -online

• This will take you to https://api.itglue.com/developer/#accounts-users

  I believe I have everything in a good initial state but with writing some much documentation it wouldn't surprise me if there are grammar or consistency issues between the wrappers. I have tested most of the functions after the modifications and I have not found any issues yet but I am only familiar with GET methods right now.

What are everyone's thoughts on the structure and or next steps?

• Is there a better way to relay\show JSON examples in the help comments?

Unless otherwise defined all modifications are additions of comment-based help

0. Pester Tests

   • Updated to Pester 5
   • General test improvements (could use some more tweaks)
   • Added HelpComment.Tests to help validate help-based comments

1. APIKey

   • Added cmdletbinding to all functions
   • Fixed basic scriptAnalyzer warnings
   • New Test-ITGlueAPIKey
   • Added because its helpful when needing to validate general functionality or when using RMM deployment tools

2. BaseURI

   • Added cmdletbinding to all functions
   • Fixed basic scriptAnalyzer warnings

3. ModuleSettings

   • Added cmdletbinding to all functions
   • Fixed basic scriptAnalyzer warnings
   • New Remove-ITGlueModuleSettings
   • Nice to remove saved settings from systems when no longer needed

4. ConfigurationInterfaces

   • Added cmdletbinding to New-ITGlueConfigurationInterfaces

5. Configurations

   • Added cmdletbinding to New-ITGlueConfigurations

6. ConfigurationStatues

   • Added cmdletbinding to New-ITGlueConfigurationStatuses
   • Added cmdletbinding to Set-ITGlueConfigurationStatuses

7. ConfigurationTypes

   • Added cmdletbinding to New-ITGlueConfigurationTypes
   • Added cmdletbinding to Set-ITGlueConfigurationTypes

8. Contacts

   • Added cmdletbinding to New-ITGlueContacts

9. ContactTypes

   • Added cmdletbinding to New-ITGlueContactTypes
   • Added cmdletbinding to Set-ITGlueContactTypes

10. FlexibleAssetFields

    • Added cmdletbinding to New-ITGlueFlexibleAssetFields
    • Might need to add in mandatory parameters to Get-ITGlueFlexibleAssetFields (Can come later)

11. FlexibleAssetTypes

    • Added cmdletbinding to New-ITGlueFlexibleAssetTypes
    • Added cmdletbinding to Set-ITGlueFlexibleAssetTypes

12. FlexibleAssets

    • Might need to add mandatory parameter's to Get-ITGlueFlexibleAssets (Can come later)

13. Locations

    • Added cmdletbinding to New-ITGlueLocations
    • Should the "Remove-ITGlueLocations" data param need an update paramSet?

14. Manufacturers

    • Added cmdletbinding to New-ITGlueManufacturers
    • Added cmdletbinding to Set-ITGlueManufacturers

15. Models

    • Added cmdletbinding to New-ITGlueModels

16. Organizations

    • Added cmdletbinding to New-ITGlueOrganizations

17. OrganizationStatuses

    • Added cmdletbinding to New-ITGlueOrganizationStatuses
    • Added cmdletbinding to Set-ITGlueOrganizationStatuses

18. OrganizationTypes

    • Added cmdletbinding to New-ITGlueOrganizationTypes
    • Added cmdletbinding to Set-ITGlueOrganizationTypes

19. PasswordCategories

    • Added cmdletbinding to New-ITGluePasswordCategories
    • Added cmdletbinding to Set-ITGluePasswordCategories

20. Passwords

    • Added cmdletbinding to New-ITGluePasswords

21. Users

    • Added cmdletbinding to Set-ITGlueUsers

[Celerium]

With all the new development going on when would be a good time to merge these changes? I was thinking once #157 is completed.

This would give me or others time to review any grammar or consistency issues with help descriptions between the functions as well as give a good jumping-off point for further module development.

[Celerium]

I've updated various, grammar, spelling, and word consistency for each of the functions and feel like this is a good starting point for the modules about_help.

I can switch this to ready for review so it can be merged now or after #157.

[adrianwells]

@Celerium this is an impressive amount of work as @CalebAlbers noted. It seems it should be merged soon. @davidhaymond thoughts?

[Celerium]

@Celerium this is an impressive amount of work as @CalebAlbers noted. It seems it should be merged soon. @davidhaymond thoughts?

@adrianwells thank you, something to note as it's been a while, so it would be a good idea to do a once-over of this addition as well as make sure any of the new functions include comment-based help.

[davidhaymond]

@Celerium this is an impressive amount of work as @CalebAlbers noted. It seems it should be merged soon. @davidhaymond thoughts?

I would love to see this merged and included in v2.3.0. I'll see if I can find time this week to review it.

[davidhaymond]

@Celerium This PR seems ready to me. Could you rebase your branch on top of the current master branch and squash all the commits? I'd like to try to keep the Git history relatively clean.

It looks like you've allowed maintainers to edit the PR, so if you like, I can rebase and merge for you.

[Celerium]

@Celerium This PR seems ready to me. Could you rebase your branch on top of the current master branch and squash all the commits? I'd like to try to keep the Git history relatively clean.

It looks like you've allowed maintainers to edit the PR, so if you like, I can rebase and merge for you.

@davidhaymond, if you could that would be wonderful. Thank you

[davidhaymond]

I've cleaned up this PR and merged into master! Thanks again for your incredible contribution, @Celerium!