Skip to content

Conversation

@yaira2
Copy link
Member

@yaira2 yaira2 commented Nov 14, 2025

Resolved / Related Issues

To prevent extra work, all changes to the Files codebase must link to an approved issue marked as Ready to build. Please insert the issue number following the hashtag with the issue number that this Pull Request resolves.

Steps used to test these changes

Stability is a top priority for Files and all changes are required to go through testing before being merged into the repo. Please include a list of steps that you used to test this PR.

  1. Set Files as default file manager
  2. Changed language to not match the Windows language
  3. Fully closed Files
  4. Opened Recycle Bin shortcut from desktop

@yaira2 yaira2 requested a review from gave92 November 14, 2025 02:25
@yaira2
Copy link
Member Author

yaira2 commented Nov 14, 2025

@gave92 this PR reverts a change you made in #6293. Do you know if this brings back the underlying issue it was originally supposed to solve (I wasn't able to reproduce this issue on my side)?

@yaira2 yaira2 added the ready for review Pull requests that are ready for review label Nov 14, 2025
@yaira2 yaira2 requested a review from hishitetsu November 14, 2025 02:27
@gave92
Copy link
Member

gave92 commented Nov 18, 2025

Hey yair, it seems like removing the code above does not cause #6190 to come back 👍. That said I cannot reproduce issue #15083 so not sure if removeing this fixes the issue. Also you may want to remove the similar logic from InitializeApplicationAsync

// Convert short name to long name (#6190)
unescapedValue = folder.Path;

@yaira2
Copy link
Member Author

yaira2 commented Nov 18, 2025

@gave92 thank you for checking. The testing steps mentioned in the PR description seem to be an easy way to reproduce the issue.

I'm hesitant to remove the code from InitializeApplicationAsync unless we know of a scenario where it causes issues. Do you know when the line was first added?

@yaira2 yaira2 added ready to merge Pull requests that are approved and ready to merge and removed ready for review Pull requests that are ready for review labels Nov 18, 2025
@yaira2 yaira2 merged commit 80a5f59 into main Nov 18, 2025
9 checks passed
@yaira2 yaira2 deleted the ya/FixOpenShellFolders branch November 18, 2025 21:20
gave92 added a commit to gave92/files-uwp that referenced this pull request Nov 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready to merge Pull requests that are approved and ready to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: Not a folder error when opening shell addresses

3 participants