Skip to content

Update UWP Assets Redirect to v1.1#4056

Merged
m417z merged 9 commits into
ramensoftware:mainfrom
atferrys:main
May 15, 2026
Merged

Update UWP Assets Redirect to v1.1#4056
m417z merged 9 commits into
ramensoftware:mainfrom
atferrys:main

Conversation

@atferrys
Copy link
Copy Markdown
Contributor

@atferrys atferrys commented May 13, 2026

Changelog

If this pull request updates an existing mod, describe the changes below:

  • Added support for .ico redirections:
    • You can now provide the path to an .ico file, and assets will be automatically generated,
    • You can find more information in the mod's details tab under "Applying redirections".
  • Fixed WindowsApps read permissions that caused the mod to hang on loading
  • Added dialogs during Icon themes download or when an error occurs
  • Added "Luna" theme

Mod authorship

If this pull request introduces a new mod, please complete the section below.

This mod was created by:

    • The submitter, without AI assistance
    • The submitter, with AI assistance
    • Claude
    • ChatGPT
    • Gemini
    • Another AI (please specify):
    • Other (please specify):

Please select the options that best apply. Your selection does not affect the acceptance criteria, but it helps reviewers understand the context of the code and provide relevant feedback.

@atferrys atferrys marked this pull request as draft May 14, 2026 15:24
@m417z
Copy link
Copy Markdown
Member

m417z commented May 14, 2026

Ping me when it's ready. I don't think GitHub notifies when the draft state changes.

@atferrys atferrys marked this pull request as ready for review May 14, 2026 19:31
@atferrys
Copy link
Copy Markdown
Contributor Author

atferrys commented May 14, 2026

@m417z should be ready

Comment thread mods/uwp-assets-redirect.wh.cpp
Comment thread mods/uwp-assets-redirect.wh.cpp Outdated
@atferrys atferrys requested a review from m417z May 15, 2026 11:36
@m417z m417z merged commit 7480b0e into ramensoftware:main May 15, 2026
3 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.

2 participants