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

Add logic to remove Registry Keys via Tweaks #2837

Merged

Conversation

Marterich
Copy link
Contributor

Type of Change

  • New feature

Description

Adds the option to specify "" as a Registry Value in the tweaks.json registry definition.
When this is provided, the value will be fully removed instead of simply being set to a specific value.

Testing

Tested this on a demo Tweak and it seemed to work.

Impact

This should not affect anything except when the Registry value is explicitly set to

Issue related to PR

Additional Information

@MyDrift-user I have no clue where to put this in the documentation. Maybe you can give me a hint on where to mention this "feature"

@MyDrift-user
Copy link
Contributor

Idk, maybe we should generally improve the contribution guide (in another PR), like mention the templated way of doing reg changes, task changes, service changes, explain invokescript and undoscript and so one..

@ChrisTitusTech ChrisTitusTech merged commit d855b08 into ChrisTitusTech:main Oct 1, 2024
1 check passed
@Marterich Marterich deleted the remove-registry-entry branch October 3, 2024 10:59
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.

Fix "Managed by Organisation"
3 participants