These are chat archives for reactioncommerce/reaction

5th
Apr 2017
xuanvinhvu
@xuanvinhvu
Apr 05 2017 00:52

I tried out installing a custom theme as suggested here https://github.com/reactioncommerce/reaction-example-theme

After the installation, I stop and start reaction but I dont see anything changed. The navbar-default-bg supposed to be red but I’m not sure where I should look for it? How do I know that it works?

Brent Hoover
@zenweasel
Apr 05 2017 00:55
Testing it out now
Brent Hoover
@zenweasel
Apr 05 2017 01:01
REACTION.png
so this is what I am seeing
xuanvinhvu
@xuanvinhvu
Apr 05 2017 01:08
then i didnt install the plugin correctly :)
Brent Hoover
@zenweasel
Apr 05 2017 01:08
where did you clone it into?
xuanvinhvu
@xuanvinhvu
Apr 05 2017 01:09
i downloaded the zip
and then add the files into my project
Screen Shot 2017-04-04 at 9.09.56 PM.png
Brent Hoover
@zenweasel
Apr 05 2017 01:16
well a) good on you for using Webstorm
and b) I think you need to create a directory
reaction_-____Projects_js_reaction_.png
xuanvinhvu
@xuanvinhvu
Apr 05 2017 01:18
aha! let me try, thanks!
xuanvinhvu
@xuanvinhvu
Apr 05 2017 01:26
Great it’s working. Thanks again.
Brent Hoover
@zenweasel
Apr 05 2017 01:26
:+1:
Richa Bhardwaj
@richa
Apr 05 2017 05:18

hi @zenweasel I'm still getting the same error:
=> Started proxy.
=> Started MongoDB.
=> Errors prevented startup:

While processing files with ecmascript (for target web.browser):

/Users/richabhardwaj/.meteor/packages/ecmascript/.0.6.3.v8afs8++os+web.browser+web.cordova/plugin.compile-ecmascript.os/npm/node_modules/meteor/babel-compiler/node_modules/babel-core/lib/transformation/plugin.js:131:13:
Plugin 0 specified in "foreign" provided an invalid property of
"wrapped"
at Plugin.init

Brent Hoover
@zenweasel
Apr 05 2017 06:01
@richa I don’t know what the issue could be. I just tried it from a fresh checkout and did not see that error
something is failing in the meteor npm install stage.
Can you give me the results of reaction -v?
Richa Bhardwaj
@richa
Apr 05 2017 09:33
@zenweasel reaction -v looks like this:
Node: 7.5.0
NPM: 4.2.0
Docker: 1.13.1
Reaction CLI: 0.7.9
Reaction: 1.1.0
Richa Bhardwaj
@richa
Apr 05 2017 10:07
@zenweasel I managed to run but it's a weird fix
I had to replace bcrypt to bcryptjs in package.json and run meteor npm install
after that the server started on port 3000
Brent Hoover
@zenweasel
Apr 05 2017 10:09
Do you have the Xcode, etc. installed?
Normally it should just fall back to the JS implementation if it can’t install the binary version
Ali S. Veliyev
@avengeralive
Apr 05 2017 10:32
Hello. I have a touble with installation of RC
I make command npm install -g reaction-cli
fixed
sorry
no, not fixed
Error: Node modules were not successfully installed. Exiting.
why this error appears?
/bin/sh: 1: meteor: not found
Richa Bhardwaj
@richa
Apr 05 2017 11:03
@zenweasel yes, I do have Xcode. without that, I can just watch movies on my machine :)
@zenweasel any suggestions for me to use if I try to deploy the app on heroku?
Brent Hoover
@zenweasel
Apr 05 2017 11:10
@richa We don’t support Heroku deployment, but there is this guide that a community member wrote: https://medium.com/@oreagunbiade/reactioncommerce-heroku-deploy-guide-f5672e242863
Richa Bhardwaj
@richa
Apr 05 2017 11:12
thx @zenweasel
Brent Hoover
@zenweasel
Apr 05 2017 11:13
@avengeralive Sounds like you may have an issue with your installation of Node?
Maybe try uninstall/reinstalling?
Ali S. Veliyev
@avengeralive
Apr 05 2017 11:18

You are attempting to run Meteor as the 'root' superuser. If you are
developing, this is almost certainly not what you want to do and will likely
result in incorrect file permissions. However, if you are running this command
in a build process (CI, etc.), or you are absolutely sure you know what you are
doing, set the METEOR_ALLOW_SUPERUSER environment variable or pass
--allow-superuser to proceed.

Even with METEOR_ALLOW_SUPERUSER or --allow-superuser, permissions in your app
directory will be incorrect if you ever attempt to perform any Meteor tasks as
a normal user. If you need to fix your permissions, run the following command
from the root of your project:

sudo chown -Rh <username> .meteor/local

now I have this problem
@zenweasel I fixed previous
Brent Hoover
@zenweasel
Apr 05 2017 11:43
@avengeralive Like the message says, you don’t want to be running Reaction as a superuser
besides not working, it’s a security risk
Marvin Edwin Sena Ortiz
@Marvinh5
Apr 05 2017 15:19
h
Hi
does reaction commerce include auctions?
if not is there a way to add auctions ?
Carlos Olivera Terrazas
@carlos-olivera
Apr 05 2017 17:22
Hi! I had a very high CPU peaks with product photos stored in Fs.Collection and GridFS, (by default in reactioncommerce), I had to move to filesystem and the problem was gone, someone with similar experience?
Lorenzo Campanis
@lcampanis
Apr 05 2017 17:59
This is interesting, as we are working on exactly the same problem at the moment. Server crashes when photos are uploaded fast. We do have a few custom implementations of image processing on top, but managed to crash RC's v1.0 as well. Is there something you can share as we were just discussing moving towards https://github.com/vsivsi/meteor-file-collection. Any input would be appreciated
Spencer Norman
@spencern
Apr 05 2017 18:00
How many photos are you uploading at a time when this is happening?
Lorenzo Campanis
@lcampanis
Apr 05 2017 18:01
we only allow 1 upload at a time per user, we multiplied this by a few concurrent users and down it went..
problem is with collectionFS not RC
Spencer Norman
@spencern
Apr 05 2017 18:02
Multiple instances of the app or a single instance?
Lorenzo Campanis
@lcampanis
Apr 05 2017 18:03
single instance of the app. It seems that collectionFS is sleeping on fixes for quite a long time. So new modules for storing files have come out which solve the problem with readstream locking
Spencer Norman
@spencern
Apr 05 2017 18:09
hmm, so does it look like using a forked version of collectionFS with fixes pulled in solves this?
Lorenzo Campanis
@lcampanis
Apr 05 2017 18:12
vsivsi isn't exactly a fork, is this what you mean? What GridFS isn't implementing is a lock on read/write, everything else is fine. So uploading or downloading is inevitably crushing the container
Spencer Norman
@spencern
Apr 05 2017 18:15
yeah, that’s what I was asking about. So CPU spikes on upload and too many concurrent crashes container because of GridFS readstream lock?
Is #2071 describing the same issue?
Lorenzo Campanis
@lcampanis
Apr 05 2017 18:22
That's related yes, it will crush in various ways but they all lead to GridFS not handling either its deletes from _tempstore correctly, the one you sent or a band new one we just got now
-----------FINISH STREAM _tempstore
---SA REMOVE
/opt/reaction/dist/bundle/programs/server/node_modules/fibers/future.js:280
                        throw(ex);
                        ^

TypeError: Cannot read property 'keys' of undefined
    at packages/cfs_tempstore.js:326:63
    at runWithEnvironment (packages/meteor.js:1176:24)
Spencer Norman
@spencern
Apr 05 2017 18:23
Cool. I’m going to put some of this chat into that issue to keep track of it
Lorenzo Campanis
@lcampanis
Apr 05 2017 18:25
No probs.. We investigated and hacked GridFS and tempstore but found no way for GridFS to handle concurrent uploads correctly. It will pass 1/10 times
And after reading everything we could find and attempting to work it directly from collectionFS with a custom implementation, the outcome is simply that GridFS is unable to handle concurrent uploads correctly
Aaron Judd
@aaronjudd
Apr 05 2017 18:28
we were testing the S3 plugin, I’m wondering if you only used S3 and not GridFS, does collectionFS still do the chunking/uploads -> I’m assuming that’s not the case, that’d it’d alleviate the GridGS issue (while still writing metadata to the Media collection)
Lorenzo Campanis
@lcampanis
Apr 05 2017 18:30
yeah that's probably the best solution, we were leaning towards S3 as well, the chunks are what cause the issue in all cases
Spencer Norman
@spencern
Apr 05 2017 18:34
Does the vsivsi/meteor-file-collection package solve the problems? Have you tested with it?
Aaron Judd
@aaronjudd
Apr 05 2017 18:34
my inclination is that GridFS isn’t the real source, but CFS. We’ve been discusing for a while, forking CFS and fixing some of the issue, but that’s been delayed by the competing desire to just replace CFS -> (and keep the Media collection / metadata compatible layer)
Lorenzo Campanis
@lcampanis
Apr 05 2017 18:37
Yep agreed! Problem is that S3 currently isn't complete, and not sure where it would hit us.. There are other solutions but all need quite a lot of time to implement based on the current structure
https://github.com/VeliovGroup/Meteor-Files
https://github.com/vsivsi/meteor-file-collection
Carlos Olivera Terrazas
@carlos-olivera
Apr 05 2017 18:42
I am using cfs:filesystem, no CPU peaks (at leas not so critic as GridFS), but I have an issue with Blaze, after reboot the container I got a net::ERR_CONTENT_LENGTH_MISMATCH error, only in Blaze list products templates (not in React detail product) any thoughts?
Carlos Olivera Terrazas
@carlos-olivera
Apr 05 2017 18:47
After delete and upload again the same photo (for all products), all are ok, until reboot the container again. and the list (toGrid photo) get error.
Lorenzo Campanis
@lcampanis
Apr 05 2017 18:50
Yeah cfs is most definitely the issue, S3 is not covering it CollectionFS/Meteor-CollectionFS#941
Marvin Edwin Sena Ortiz
@Marvinh5
Apr 05 2017 20:31
nice, that's what i call beign ignored
Bible Logos The Game
@biblelogosthegame
Apr 05 2017 20:59
@zenweasel Do you have any advice for me? I've been having this "data you sent is not accepted as valid" error when I try to check out. I'm testing Reaction for an online store, but for several days I haven't been able to find any help on this error https://files.gitter.im/reactioncommerce/reaction/OYwg/address.PNG
Brent Hoover
@zenweasel
Apr 05 2017 21:04
I can take a look at it, but I am on a conference call right now, so give me a little bit
Brent Hoover
@zenweasel
Apr 05 2017 22:04
@biblelogosthegame Can you look at your server output and see if you see any errors?
Spartano
@Spartano
Apr 05 2017 23:06
hello, i am tryng to set up the tutorial on creatng a custom them and i dont understand some things. Anione has 5 mins to guide me?
never mind, i figure it out :)
Brent Hoover
@zenweasel
Apr 05 2017 23:14
:+1: