-
Notifications
You must be signed in to change notification settings - Fork 38
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
[MS] Workspace history #8743
[MS] Workspace history #8743
Conversation
f0e7b91
to
68f042c
Compare
f89a963
to
a999559
Compare
9c86f7f
to
d5e30f7
Compare
60caa71
to
fd34313
Compare
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.
I've performed some quick tests and it seems to be working OK. IMO this feature needs some UI/UX tweaking, but it is probably better to do that in another PR (also to take the time to use this feature).
Just some comments on the wording.
I did not review any ts/vue file, so better wait for other approvals before merge.
25f1e69
to
efcbf4b
Compare
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.
@Max-7 please add a newsfragment for the changelog
24b8dca
to
f9f7c72
Compare
Co-authored-by: fabienSvstr <[email protected]>
Co-authored-by: Marcos Medrano <[email protected]>
Co-authored-by: NicoTuxx <[email protected]>
f9f7c72
to
a350b78
Compare
Closes #8663
Waiting for #8662