microsoft / ReportingServicesTools

Reporting Services Powershell Tools
MIT License
458 stars 213 forks source link

[Write-RsRestCatalogItem] add `Hidden` parameter #389

Closed zsvr-pkruse closed 2 years ago

zsvr-pkruse commented 2 years ago

Fixes absence of Hidden parameter for Write-RsRestCatalogItem

Changes proposed in this pull request:

How to test this code:

Has been tested on (remove any that don't apply):


Details

The following can now be uploaded as hidden:

Clarify documentation of

to be ignored by large Power BI Report files.

Add documentation for new Hidden parameter. Add example for new Hidden parameter. Fix some formatting issues. Add information about ignored parameters to verbose write. Remove ignored fields from large file upload. See https://app.swaggerhub.com/apis-docs/microsoft-rs/pbirs/2.0#/PowerBIReports/UploadPowerBIReport Make Try upload block smaller. The follow-up catch block was still relying on parts of the now moved out code block. Overwrite upload code now uses same $requestBody as normal upload. Kpi Hidden property is only changed if parameter Hidden is used.

Automated tests weren't extended because Description paramater isn't tested either.

Please let me know how to improve this Pull Request.

zsvr-pkruse commented 2 years ago

@microsoft-github-policy-service agree [company="Stiftung Zentrale Stelle Verpackungsregister"]

zsvr-pkruse commented 2 years ago

@microsoft-github-policy-service agree company="Stiftung Zentrale Stelle Verpackungsregister"

zsvr-pkruse commented 2 years ago

Build failed because an external server is not available during testing? I'm not sure how to proceed here.

VERBOSE: POST http://localhost/reports/api/v1.0/ReportServerInfo/Model.UpdateSettings with -1-byte payload
System.Net.WebException: The remote server returned an error: (503) Server Unavailable.
   at Microsoft.PowerShell.Commands.WebRequestPSCmdlet.GetResponse(WebRequest request)
   at Microsoft.PowerShell.Commands.WebRequestPSCmdlet.ProcessRecord()