We want to support desktop device types to be able to handle native notifications. This diff adds macos and windows to the possible platform and sets config on the web app depending on where it's running. Older desktop clients with newer web app will still be treated as web. Additionaly if on the socket initialization we detect that someone has a user agent with "Electron" and still has device token of web we will ask the web app to send us the current device type, so already logged in users will be updated.
Details
Details
- check that after login the device type is macos
- check that for already logged in users the device type changes to macos and remains that after logging out on the anoonymous cookie
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
| keyserver/src/socket/socket.js | ||
|---|---|---|
| 460 ↗ | (On Diff #23066) | I might be wrong, but I think if viewer.userAgent is not set, you'll need optional chaining on the function invocation as well |
| keyserver/src/socket/socket.js | ||
|---|---|---|
| 460 ↗ | (On Diff #23066) | I tested it and it seems that if viewer.userAgent isn't set the call expression will evaluate to undefined. Flow also types it as void | boolean |
| keyserver/src/socket/socket.js | ||
|---|---|---|
| 460 ↗ | (On Diff #23066) | Thanks for checking! |