Skip to content
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

[1.4.0] No Notifications? #473

Closed
oitsjustjose opened this issue Dec 3, 2016 · 16 comments
Closed

[1.4.0] No Notifications? #473

oitsjustjose opened this issue Dec 3, 2016 · 16 comments

Comments

@oitsjustjose
Copy link

So, I literally installed YakYak today for the first time and so far I'm incredibly pleased!! The one thing I can't figure out is why none of the notifications are working. Any thoughts?

Here's what my notifications settings are currently at:

ss 2016-12-02 at 11 08 03

Any suggestions would be awesome. Thanks!!

@davibe
Copy link
Contributor

davibe commented Dec 3, 2016

maybe your conversation is muted ? can someone else test this on windows ?

@oitsjustjose
Copy link
Author

Nope, I checked, all conversations I'm receiving messages on are set to receive notifications.

@averissimo
Copy link
Member

averissimo commented Dec 3, 2016

Working in Windows 7 32-bit

selection_00285

@averissimo
Copy link
Member

but from windows 8 and above a different system is used

@davibe
Copy link
Contributor

davibe commented Dec 3, 2016

so basically node-notifier is not compatible with windows 8+ ?

@averissimo
Copy link
Member

It should be, it just uses a different notification system

@oitsjustjose
Copy link
Author

So 1.4.1 doesn't seem to work either, on neither my laptop nor my desktop. In fact, the notifications don't work for my roommate on 1.4.0 on OS X either - are you supposed to install something else before installing YakYak? I feel like we're missing a dependency, but I've installed NodeJS.

@averissimo
Copy link
Member

averissimo commented Dec 5, 2016

YakYak is self contained, you should only need the contents of the zip file.

Can you open the Inspector (via YakYak menu, or Ctrl+Alt+i) and see if there are any errors?

@mliq
Copy link
Contributor

mliq commented Dec 5, 2016

Just want to chime in to confirm notifications work fine for me on OSX 10.11.6, YakYak 1.4.0.

I do have Show username and Show message both unchecked.

@oitsjustjose
Copy link
Author

oitsjustjose commented Dec 5, 2016

Not seeing any errors. Unless this is different from Chrome's usual inspector, the Console isn't throwing any errors when receiving a message.

capture

ss 2016-12-05 at 02 56 50

@averissimo
Copy link
Member

No errors :\

I have no clue on what this could be. Can you confirm or test the following?

1 - Open the Inspector, switch for the console tab and write/paste the following:

require('node-notifier').notify('Something should appear')

It should create a notification, and please copy here the result.

2 - Are you always using the same account to login?

3- Are the notifications "On" in the webinterface of hangouts? (see image below)

outrz95xr313w

ps. thanks @mliq

@oitsjustjose
Copy link
Author

oitsjustjose commented Dec 5, 2016

No dice

require('node-notifier').notify('Something should appear')

WindowsToaster {options: Object, domain: null, _events: EventHandlers, _eventsCount: 0, _maxListeners: undefined}Growl: Growl(options)Notification: WindowsToaster(options)NotificationCenter: NotificationCenter(options)NotifySend: NotifySend(options)WindowsBalloon: WindowsBalloon(options)WindowsToaster: WindowsToaster(options)_events: EventHandlers_eventsCount: 0_maxListeners: undefineddomain: nulloptions: Object__proto__: EventEmitter

@averissimo
Copy link
Member

averissimo commented Dec 5, 2016

run:

require('node-notifier').notify('caca', function(err){console.log('err', err);})

Can you take a screenshot of the result?

Then run

new require('node-notifier').WindowsToaster({fallback: true}).notify('caca', function(err){console.log('err', err);})

screenshot again :)

What exactly is you OS? if windows 10 try:

image

then try all this an screenshot it please

new require('node-notifier').Growl({fallback: true}).notify('caca', function(err){console.log('err', err);})
new require('node-notifier').WindowsBalloon({fallback: true}).notify('caca', function(err){console.log('err', err);})
new require('node-notifier').NotifySend({fallback: true}).notify('caca', function(err){console.log('err', err);})
new require('node-notifier').NotificationCenter({fallback: true}).notify('caca', function(err){console.log('err', err);})

@oitsjustjose
Copy link
Author

Wow. It took me 3 days to figure out what it was.

This is the worst ID10T error I think I've ever had.

I thought you were creating your own notifications, not using the built-in Windows ones. I keep those off normally (now enabled and working fine)....

So yeah. This was completely my fault. Notifications work as intended... just maybe slap this in the README? If I had paid any attention to how this was supposed to be working, I would've been able to save you the time. Sorry :|

@averissimo
Copy link
Member

LOOOOOOOL

Would this be cool?

image

@oitsjustjose
Copy link
Author

That would be perfect.

this.facepalm()

averissimo added a commit to averissimo/yakyak that referenced this issue Dec 6, 2016
* regression-211:
  removes unnecessary arg
  imposes sequence order in gulp tasks
  partially reverts feeabb in order to hide menu bar win/linux
  note indicating that notifications are native yakyak#473
  use runSequence to compile perfore starting to deploy
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants