-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Unable to start reaction #1450
Comments
Seems to be related to #1245 and the temporary fork that we're using https://github.com/ongoworks/node-authorize-net perhaps you could try doing |
do this in project directory. |
Maybe we should just publish that to the |
@sylc and @hrath2015 can you both try the new branch I just published and let me know if the authorize-net error goes away? reaction init -b authorize-on-npm
cd reaction
reaction |
error as below: (Not in NPM registry) Shall wait and try after some time. npm ERR! 404 Not found : @reactioncommerce/authorize-net |
Yeah, just discovered that too in CI. It's because org packages are automatically set to private by default. I didn't have the issue because I have access to the org. But I apparently don't have the rights to make it public, so I'm waiting on @aaronjudd to take a look with his account. I'll let you know when it's all set. |
OK. thanks |
Nevermind. Apparently you can only set the access level at the time of publishing the package. So I had to bump the version number and republish just to set it to public. No UI available for that on npm. Nice one, npm... Give it another try. |
@zenweasel and I discussed if we should publish that as a package or use the git remote, but we'd decided we didn't want to pollute our org with a bunch of packages that we can never get rid of… this issue feels like it's more likely a network error… since npm fully supports pulling direct from the repo (and this has been in place for the last several releases). @hrath2015 have you always had an issue with this and done the manual install, or were you just making a recommendation? After the package is published on npm, it might be hard to say conclusively that the issue is solved... |
I had this issue after this change was made due to security and only on windows env. |
Interesting. I bet the older version of npm that Meteor uses has a bug with Github pulls on Windows. |
all good 👍 |
I have now an error with bcrypt, but reaction is working. It takes at least 6 mins to start but it could be linked to the bcrypt error, which i'm investigating further thanks!
when i do
Building the projects in this solution one at a time. To enable parallel build, please add the "/m" switch. gyp ERR! build error npm ERR! [email protected] install: |
U need to have python installed and try. |
@jshimko @aaronjudd I remember a discussion where we wanted to remove this bcrypt dependency and use what comes with meteor. Any update? |
I have python installed. The main issue was with node-gyp. VS2015 build tools have break the build and node-gyp needed to be upgraded to 3.4.0. I finally upgraded npm to 3.10.8, which upgraded node-gyp. and the compilation went through. It is ok now. removing bcrypt will reduce the install requirement for first time users . +1 if it can be removed. thanks |
Hi,
when i did
reaction init,
i had a successful Reaction message however some error were present as well.These errors can also be reproduced by also doing
meteor npm i
:when i start reaction with reaction, i have:
then it hangs.
i'm on windows 10. i have installed all required tools. node/win32 SSL/MS build tools/Git/Imagemagick
The text was updated successfully, but these errors were encountered: