search

pusher / atom-pair

An Atom package that allows for epic pair programming
MIT License
1.45k stars 28 forks source link

Fresh install throws pusher errors #77

Closed tarwich closed 9 years ago

tarwich commented 9 years ago

When you start a new pairing session after a fresh install, you get pusher errors.

First, I installed...

apm install atom-pair

Then opened atom Then used the "AtomPair: Start a new pairing session" command image

And I got this error image

Mac OS X 10.10.3.14D136:
  arch: x86_64
Atom:
  version: 1.0.7
  architecture: x86_64
jpatel531 commented 9 years ago

Hi there! That's bizarre, the package should be using the default Pusher keys. On the settings page for the package, do you have any keys registered?

raymondhardy commented 9 years ago

Yeah i think this is also a problem with moving the settings. You can add the default keys in. "atom-pair": pusher_app_key: "d41a439c438a100756f5" pusher_app_secret: "4bf35003e819bb138249"

You can get to the settings by clicking on the box from the plugin screen the settings are listed below

xuebingli commented 9 years ago

I am experiencing this error as well. This error still appears even after I manually add my own Pusher key and secret to the plugin's config file.

jpatel531 commented 9 years ago

Copied from #78 :

Hi there - 2.0.9 should resolve this. I realize that I was only merging the main module with the config options only when the package was activated, meaning you couldn't see them in your settings page pre-activation. It should be fine now, hopefully :/ apologies!

xuebingli commented 9 years ago

Thank you for fixing it @jpatel531

atroche commented 9 years ago

nice!

tarwich commented 9 years ago

This is fixed for me on Windows. I don't have access to my Mac until Wed.

Thanks!