Skip to content
This repository has been archived by the owner on Jul 3, 2023. It is now read-only.

Improves New-MVPContribution #47

Merged
merged 3 commits into from
Feb 28, 2021
Merged

Improves New-MVPContribution #47

merged 3 commits into from
Feb 28, 2021

Conversation

PrzemyslawKlys
Copy link
Contributor

This changes:

  • Ability to define more than one AdditionalTechnology (as the interface allows this, so does API - not sure if more than 2 are allowed so for now I've not blocked anything)
  • Improved speed of the cmdlet by caching types. Otherwise with each submits four GET queries would execute unnessecary
  • Hopefully fixes New-MVPContribution returning 500 internal server error #19 as I was getting this error for some entries until I added UTF-8 in ContentType. Not sure if it's just luck that it started to work again or it really worked but no harm in doing so.
  • Uses ConvertTo-Json from Hashtable rather than the string-based body. Easier to set things PowerShell way.

@lazywinadmin lazywinadmin merged commit baeadf9 into lazywinadmin:master Feb 28, 2021
@lazywinadmin
Copy link
Owner

Thanks @PrzemyslawKlys ! Looks great! Thanks for your contributions! :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

New-MVPContribution returning 500 internal server error
2 participants