Update to nw@0.30.2
Successor of https://gitlab.com/gitlab-org/gitter/desktop/merge_requests/201
- Update to nw@0.30.2
- Remove auto-update
- Only notify when an update is available
- Use native browser Notification API, https://github.com/nwjs/nw.js/wiki/Notification, https://developer.mozilla.org/en-US/docs/Web/API/notification
- Remove
node-notifier
dependency
- Remove
If you want to see the devtools, https://www.npmjs.com/package/nw#install-a-specific-build-type-of-nwjs
npm install nw --nwjs_build_type=sdk
If you want to see devtools console output in the CMD output, add --enable-logging
Update notes
-
Create new OAuth apps for all platforms with redirect_uri: https://gitter.im/login/desktop/callback
windows-desktop-prod-v4
osx-desktop-prod-v4
linux-desktop-prod-v4
-
Update nwapp/oauth.json
with these new apps for builds
Todo/problems
-
Figure out why focus
events aren't working, http://docs.nwjs.io/en/latest/References/Window/#event-focus- Tried upgrading to
nw@0.30.0
but no change - Related issues, - Fixed by getting rid of the iframe. Instead, we now show the Gitter app directly
-
Check that chat iframe inside Gitter itself doesn't affect things too adversely - It still affects things but now we are one level better than before
- Tried upgrading to
-
Should we wait for focus hooks to be fixed? https://github.com/nwjs/nw.js/issues/4601#issuecomment-386528893 - No, we will always show notifications regardless of focus
-
"Developer Tools" menu option fails to open devtools -
Window.isDevToolsOpen()
doesn't work, https://github.com/nwjs/nw.js/issues/4487 - We no longer toggle devtools, always just call
Window.showDeveloperTools()
-
-
Running into an issue where the app crashes when you sign in (it works fine if you sign out then sign back in) - Reproduction steps:
- Start the app as signed out,
npm start -- --base-url=http://localhost:5000 --faye-url=http://localhost:5000/bayeux --remote-debugging-port=1234
- Try signing in
- After pressing "Allow" on the decision screen the app will crash
Uncaught sendRequestSync @ extensions::sendRequest:148 (anonymous) @ extensions::nw.Window:160 (anonymous) @ extensions::binding:64 (anonymous) @ extensions::binding:377 NWWindow @ extensions::nw.Window:182 (anonymous) @ extensions::nw.Window:658 (anonymous) @ extensions::binding:64 (anonymous) @ extensions::binding:377
- Start the app as signed out,
- It seems to work just fine if when using
gitter.im
vslocalhost:5000
🤷 . I'll leave it for now
- Reproduction steps:
Notes
npm start -- --base-url=http://localhost:5000 --faye-url=http://localhost:5000/bayeux --update-url=http://localhost:5350 --remote-debugging-port=1234
npm start -- --remote-debugging-port=1234 --update-url=http://192.168.1.163:5350 --verbose --enable-logging
- Open already intalled app with args
- Windows:
"C:\Users\MLM\AppData\Local\Programs\Gitter\Gitter.exe" --remote-debugging-port=1234 --update-url=http://localhost:5350 --verbose --enable-logging
- macOs:
open /Applications/Gitter.app --args --remote-debugging-port=1234 --update-url=http://192.168.1.163:5350 --verbose --enable-logging
- Linux:
/usr/local/bin/gitter --remote-debugging-port=1234 --update-url=http://192.168.1.163:5350 --verbose --enable-logging
- Windows:
-
dataPath
user data directory, https://github.com/nwjs/nw.js/wiki/App#datapath- Windows:
C:\Users\MLM\AppData\Local\Gitter\User Data\Default\gitter_preferences.json
- Windows:
Closes https://gitlab.com/gitlab-org/gitter/desktop/issues/239, https://gitlab.com/gitlab-org/gitter/desktop/issues/231, https://gitlab.com/gitlab-org/gitter/desktop/issues/222, https://gitlab.com/gitlab-org/gitter/desktop/issues/150, https://gitlab.com/gitlab-org/gitter/desktop/issues/209
Edited by 🤖 GitLab Bot 🤖