-
Notifications
You must be signed in to change notification settings - Fork 127
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
General: Show user who created the workfile on the details pane of workfile manager #5093
General: Show user who created the workfile on the details pane of workfile manager #5093
Conversation
Can't test outside of windows at the moment, but the code is looking good and it behaves correctly in windows. Meaning nothing changes. |
Awesome, thank you @mkolar for testing! |
Co-authored-by: Jakub Trllo <43494761+iLLiCiTiT@users.noreply.github.com>
Co-authored-by: Jakub Trllo <43494761+iLLiCiTiT@users.noreply.github.com>
Co-authored-by: Jakub Trllo <43494761+iLLiCiTiT@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Don't have linux or mac to test.
I can confirm it still works with the current changes on my Linux box. Re: mac I don't have OP set up there but I could confirm that |
Thank you @fabiaserra merging. |
Changelog Description
New PR for #5087, which was closed after merging
next-minor
branch and then realizing we don't need to target it as it was decided it's not required to support windows. More info on that PR discussion.Small addition to add name of the
user
who created the workfile on the details pane of the workfile manager:Testing notes:
User
entry showing who created that file