-
-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
format current and sync client sessions in Personal #438
Conversation
@icewind1991, thanks for your PR! By analyzing the annotation information on this pull request, we identified @ChristophWurst and @PVince81 to be potential reviewers |
👍 nice |
774e77a
to
688b650
Compare
Would it be possible to have a tooltip with the full info? Or an 'extra info' thingy or whatever. Sometimes you want to know the exact fingerprint. |
Ah nevermind you did that. |
viewData.name = t('settings', 'Sync client - {os}', { | ||
os: matches[1], | ||
version: matches[2] | ||
}) |
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.
missing ;
Beside the comments above this looks awesome 👍 |
688b650
to
df2dffa
Compare
All fixed |
Tested and works 👍 |
👍 |
@icewind1991 you are an awesome dude! :) 👍 |
It took me a while to remember what "mirall" was and I doubt any regular user would know without a bit of googling.
This replaces the user agent of sync clients and the current session with a more descriptive name, full user agent can still be found in the tooltip
Before:
After
Further formatting (show nicer browser names instead of their full user agent) can be added at a later date
cc @nextcloud/designers