Skip to content
This repository has been archived by the owner on Dec 27, 2022. It is now read-only.

Change parameter value #14

Closed
fmigacz-zz opened this issue Dec 29, 2020 · 1 comment 路 Fixed by #8
Closed

Change parameter value #14

fmigacz-zz opened this issue Dec 29, 2020 · 1 comment 路 Fixed by #8
Assignees
@DeemOnSecurity
Labels
documentation This issue or pull request improves or adds to documentation

Comments

@fmigacz-zz
Copy link

馃殌 Feature Proposal

In Export-UALData function, change "Sharepoint" value of -WorkloadType parameter to "SharePoint"

  • function call/parameter passing on line 143
  • use of parameter value on line 324

Motivation

"Sharepoint" is an incorrect capitalization of the service. A PR was raised to fix this in the comments (#13), but the repo owners should make sure any parameter value changes have no additional effect.

Example

Minor typographical change.

Pitch

This is only to correctly identify the queried service in Microsoft M365.
@DeemOnSecurity DeemOnSecurity added the documentation This issue or pull request improves or adds to documentation label Dec 29, 2020
@DeemOnSecurity
Copy link
Contributor

Thank you. We will address this when we merge pull request #8

@DeemOnSecurity DeemOnSecurity self-assigned this Dec 29, 2020
@DeemOnSecurity DeemOnSecurity linked a pull request Dec 29, 2020 that will close this issue
Various Improvements / Fixes #8
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@DeemOnSecurity DeemOnSecurity

Labels
documentation This issue or pull request improves or adds to documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Various Improvements / Fixes
2 participants
@fmigacz-zz @DeemOnSecurity