Skip to content

UX improvements#1479

Merged
bill-long merged 1 commit into
mainfrom
bilong-logcollector
Feb 8, 2023
Merged

UX improvements#1479
bill-long merged 1 commit into
mainfrom
bilong-logcollector

Conversation

@bill-long
Copy link
Copy Markdown
Member

@bill-long bill-long commented Feb 8, 2023

ExchangeLogCollector does some things that negatively impact the user experience. This PR:

  • Removes the 1-second delay. We shouldn't pause for no reason.
  • Removes Clear-Host. We shouldn't clear the display for no reason.
  • Removes the EULA prompt. Other more invasive scripts do not have EULA prompts, so why does this one?

@bill-long
UX improvements
88e04a4 
- Remove 1-second delay
- Remove Clear-Host
- Remove EULA prompt
@bill-long bill-long requested a review from a team as a code owner February 8, 2023 16:18
@bill-long bill-long merged commit f6fdeaf into main Feb 8, 2023
@bill-long bill-long deleted the bilong-logcollector branch February 8, 2023 16:28
@dpaulson45 dpaulson45 mentioned this pull request Apr 11, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dpaulson45 dpaulson45 dpaulson45 approved these changes

Assignees

No one assigned

Labels

Enhancement New feature or request Exchange Log Collector

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bill-long @dpaulson45