microsoft/PowerShellForGitHub

Update Invoke-GHRestMethodMultipleResult to take a body

Offen

#290 geöffnet am 27.09.2020

 (6 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)PowerShell (189 Forks)auto 404
enhancementgood first issuehelp wantedup for grabs

Repository-Metriken

Stars
 (635 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

Feature Idea Summary

When writing up the workaround for #289, I realized that there are GET API's which may have multiple pages of results where we need to send a body. Therefore, Invoke-GHRestMethodMultipleResult needs to be updated to take in a body, just like Invoke-GHRestMethod.

Feature Idea Additional Details

An alternate approach to this would be to make Invoke-GHRestMethod the common entry-point for both single and multi-page results, and have its internal implementation simply call out to two different internal-only helper methods. Get methods could always call into the "multi-result" internal helper, because that would work fine even for a result with a single-page.

Requested Assignment

I'm leaving this open for someone else to tackle. If no-one expresses interest in the next week or so, I'll take it on myself.

Operating System

OsName               : Microsoft Windows 10 Pro
OsOperatingSystemSKU : 48
OsArchitecture       : 64-bit
WindowsVersion       : 2004
WindowsBuildLabEx    : 19041.1.amd64fre.vb_release.191206-1406
OsLanguage           : en-US
OsMuiLanguages       : {en-US}

PowerShell Version

Name                           Value
----                           -----
PSVersion                      5.1.19041.1
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.19041.1
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           1.1.0.1

Module Version

Running: 0.15.1
Installed:

Contributor Guide