Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Clean up network profiles #259

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

Clean up network profiles #259

wants to merge 11 commits into from

Conversation

itiligent
Copy link
Contributor

(Resubmitted do to an issue with the copyright not adding somehow)

As requested by @oskardotglobal. This PR will help keep Windows networking tidy. The install.bat changes will:

Create a new powershell script and save this to c:\windows. The Powershell script will clear unused network profile names that were automatically created at previous system reboots.
Install.bat also creates a new scheduled task that will run the cleanup powershell script at boot
A registry change is further added to RDPapps.reg to disable the "Do you want your PC to be discoverable" nag screen after each reboot
I've also added simple comments to each of the reg file entries for clarity.

itiligent and others added 11 commits September 18, 2024 14:50
Keep Windows networking tidy. At boot a Powershell script will run that  clears out old network profile names automatically created at previous system reboots. A regist change disables the "Do you want your PC to be discoverable" nag screen after eachc reboot
@oskardotglobal
Copy link
Member

great, now CI passes but still doesn't do anything

Copy link
Member

@KernelGhost KernelGhost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems fine based on a cursory look. Thanks for the changes!

@itiligent
Copy link
Contributor Author

You're welcome. Its just a small quality of life change that also makes the WinApps OOB experience feel a bit nicer. Its been tested in multiple settings and runs fine.

@itiligent itiligent closed this Sep 19, 2024
@itiligent itiligent reopened this Sep 19, 2024
@itiligent
Copy link
Contributor Author

sorry, meant to comment not close! I need my glasses!

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

Successfully merging this pull request may close these issues.

3 participants