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

Global Source Control Refresh only check local status #115

Closed
SRombautsU opened this issue Oct 19, 2022 · 2 comments
Closed

Global Source Control Refresh only check local status #115

SRombautsU opened this issue Oct 19, 2022 · 2 comments
Assignees
Milestone

Comments

@SRombautsU
Copy link
Collaborator

SRombautsU commented Oct 19, 2022

"Refresh" calls in "cm status" for all the assets in the repository, but not all the additional commands it would nee to retrieve extra info like the locks and the detection of changes

  1. If I checkout or revert some assets from the Plastic SCM GUI, using the Source Control -> "Refresh" in Unreal will correctly update the local "status" of the files (it uses "cm status" for that)
  2. But
    1. if someone else checkout & lock some assets on another workspace, using this "Refresh" won't check all the locks on the server, as an optimization to avoid too much network calls
      1. this could in fact be improved using a separate "cm lock list" command, so I'll fill a task for that
    2. if someone else submit a newer revision of an asset, using this "Refresh" won't actually check what are the latest revisions for all assets as ab optimization
      1. there is no good way to do this, it needs to be local & focused to the normal refresh of the Content Browser current view (even though we could offer an option usable on small projects)
@SRombautsU SRombautsU self-assigned this Oct 19, 2022
@SRombauts
Copy link
Owner

Note that the Plastic SCM GUI uses a "merge" command to check periodically if there are any incoming changes, which is something that would make sense to use instead of expansive calls to fileinfo

SRombauts pushed a commit that referenced this issue Apr 16, 2024
Implement a new View Changesets window with filtering capabilities
@SRombauts SRombauts added this to the 1.9.0 milestone May 28, 2024
@SRombauts
Copy link
Owner

I removed the refresh button from the global menu for release 1.9.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants