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

Fix changelist files status of files outside content folder #114

Merged
merged 4 commits into from
Apr 4, 2024

Conversation

SRombautsU
Copy link

Context: trying to checking a whole tree of private files and folders through the View Changes, see related PR #113 Fix for check in a tree of multiple private files and folders from the View Changes window in version 1.10.0

… output

This should be much faster then going through our custom cm shell parser, it remove some verbose logs and help inspect the output during development

update ParseChangelistsResults() accordingly
…RootDirs"

(other than Content, Config, Plugins, Source and Engine/Content)

The issue was trying to fill in a "map" of files grouped by Path with a variable that was "moved" using MoveTemp() so the path was cleared and the corresponding was Empty when the item was added to the map!
@SRombautsU SRombautsU added the bug label Apr 3, 2024
@SRombautsU SRombautsU self-assigned this Apr 3, 2024
@SRombautsU SRombautsU marked this pull request as ready for review April 3, 2024 10:10
@juliomaqueda juliomaqueda merged commit 6a34e65 into master Apr 4, 2024
@juliomaqueda juliomaqueda deleted the 1005172-fix-status-outside-content-folder branch April 4, 2024 08:30
@SRombautsU SRombautsU added this to the 1.10.1 milestone May 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants