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

Pass userInfo in systemPreferences.subscribeNotification #5582

Merged
merged 4 commits into from May 18, 2016

Conversation

Projects
None yet
1 participant
@zcbenz
Contributor

zcbenz commented May 18, 2016

Also rewrote the DictionaryValueToNSDictionary so it works when the dictionary includes an object that can not be converted to JSON.

Fix #5543.

@zcbenz zcbenz merged commit d2a567d into master May 18, 2016

8 of 9 checks passed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
electron-linux-arm Build #3254247 succeeded in 36s
Details
electron-linux-ia32 Build #3254248 succeeded in 32s
Details
electron-linux-x64 Build #3254249 succeeded in 110s
Details
electron-mas-x64 Build #1213 succeeded in 5 min 26 sec
Details
electron-osx-x64 Build #1215 succeeded in 6 min 25 sec
Details
electron-win-ia32 Build #223 succeeded in 6 min 3 sec
Details
electron-win-x64 Build #218 succeeded in 6 min 2 sec
Details

@zcbenz zcbenz deleted the notification-user-info branch May 18, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment