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

Fixes & updates #47

Merged
merged 5 commits into from
Apr 27, 2024
Merged

Fixes & updates #47

merged 5 commits into from
Apr 27, 2024

Conversation

dariusz-goc
Copy link
Contributor

Fixed for latest app version:

  • ChangeWindowsAppTheme.ps1
  • ChangeWindowsSystemTheme.ps1

Updated legacy script:

  • ChangeLockscreenImage.ps1

Cannot mix If statement with values, needs to run in it's own shell
Cannot mix If statement with values, needs to run in it's own shell
Updated and functional ChangeLockscreenImage.ps1 for the latest app update
@dariusz-goc
Copy link
Contributor Author

KB5036979 windows update breaks ChangeLockscreenImage.ps1, but latest update to WDD supports that function natively.

ChangeLockscreenImage.ps1 can be safely removed.

stable/ChangeLockscreenImage.ps1 Fixed Show fixed Hide fixed
stable/ChangeLockscreenImage.ps1 Fixed Show fixed Hide fixed
Copy link
Owner

@t1m0thyj t1m0thyj left a comment

Choose a reason for hiding this comment

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

@dariusz-goc Thanks for fixing these scripts!

As you mentioned the ChangeLockscreenImage script is obsolete now that WDD supports that function natively. Please remove the ChangeLockscreenImage.ps1 script and I will approve 🙂

Also, some users have reported issues with the native lockscreen functionality in WDD, please let me know if you do as well: t1m0thyj/WinDynamicDesktop#548

@dariusz-goc
Copy link
Contributor Author

I think I managed to remove the file. Please review again.

@t1m0thyj t1m0thyj merged commit a1f22ef into t1m0thyj:master Apr 27, 2024
2 checks passed
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.

None yet

2 participants