Skip to content

Refactor OneDrive uninstall handling#423

Closed
Jenso-DE wants to merge 2 commits intoRaphire:masterfrom
Jenso-DE:master
Closed

Refactor OneDrive uninstall handling#423
Jenso-DE wants to merge 2 commits intoRaphire:masterfrom
Jenso-DE:master

Conversation

@Jenso-DE
Copy link
Copy Markdown

@Jenso-DE Jenso-DE commented Jan 9, 2026

Until now, Uninstall_Microsoft_OneDrive.reg has not been used.
Now it is used for -DisableOneDrive.

Until now, uninstal.reg has not been used. 
Now it is used for -DisableOneDrive.
@Raphire
Copy link
Copy Markdown
Owner

Raphire commented Jan 14, 2026

I appreciate you taking the time to create a PR, however the -DisableOneDrive parameter is a legacy parameter that does the same as the -HideOneDrive parameter. This feature is only meant to remove the OneDrive entry from File Explorer. As such I don't think it's a good idea to change the behaviour of that parameter so drastically.

Also, as explained in my comment here the script does actually use the registry file you mentioned. It's dynamically called when you opt to remove OneDrive in Sysprep mode.

@Raphire Raphire closed this Jan 14, 2026
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.

2 participants