Skip to content

Added support for terminal notifier #22

Closed
wants to merge 5 commits into from

4 participants

@noctivityinc

Added support for outputting messages from theme watch to Mac OSX notification center. It's on by default but could be set to false with theme watch --notifiy=false

@noctivityinc

One thing you might want to change or have me change - terminal notifier gem is a dependency in my gemspec.

@hsleewis
@noctivityinc
@hsleewis
@noctivityinc

Definitely. Ok - im making some changes now. Hold on any merge

@noctivityinc

Ok - some solid progress here but one thing I simply cant figure out.

On @hsleewis advice I added control of terminal notifications to the config.yml file for the theme. Adding this line to config.yml

 :use_terminal_notifier: true

will activate terminal notifications.

Here is the ONE issue I'm having - I had to add the terminal-notifier dependency in the gemspec to make this work. No idea why but it was required. This means on OSX machines the user would need to install terminal-notifications to use this gem, which is bad.

If anyone has any idea how to fix, please let me know and Ill make the change so this can be merged.

@jduff
Shopify member
jduff commented Dec 12, 2012
@noctivityinc

Sounds good - gem name or link please...

@jduff
Shopify member
jduff commented Dec 12, 2012
@jduff
Shopify member
jduff commented May 14, 2013

Any progress on removing the hard dependency?

@noctivityinc
@jduff
Shopify member
jduff commented Jul 20, 2013

no movement, closing.

@jduff jduff closed this Jul 20, 2013
@wprater
wprater commented Dec 29, 2015

any support for this now?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.