search

evanyeung / terminal-slack

Terminal client for slack
MIT License
1.03k stars 78 forks source link

Added notifications #31

Closed alexkahn139 closed 7 years ago

alexkahn139 commented 7 years ago

Added notifications, this should work for different OS'es. I used node-notifier, wich should be installed using npm install --save node-notifier

alexkahn139 commented 7 years ago

This should also fix, some problems that happens if the user sends messages using another app or device

alexkahn139 commented 7 years ago

I Will try to fix this soon and reupload it. It was apparently my editor that did not like two spaces and changed it to four

evanyeung commented 7 years ago

Merged. Thanks for contributing!

alexkahn139 commented 7 years ago

You should maybe still change the .md file. Because you need node-notifier or does the npm install do this?

evanyeung commented 7 years ago

Since the package.json file is updated with node-notifier as a dependency, npm install will install it.