These are chat archives for reactioncommerce/reaction

7th
Mar 2017
amp9020
@amp9020
Mar 07 2017 01:37
how can you add the tracking ID for Google Analytics ?
Brent Hoover
@zenweasel
Mar 07 2017 01:38
You should be able to do it through the dashboard
amp9020
@amp9020
Mar 07 2017 01:49
@zenweasel thanks i'll double check
no looks like its in the road-line for development
Patrick Artounian
@partounian
Mar 07 2017 04:08
Any tips for better understanding file structure/location and the overall project?
Brent Hoover
@zenweasel
Mar 07 2017 05:00
I think the main thing to understand is that the file structure is a work in progress, from trying toalign with different versions of Meteor
most things are moved into plugins now, but there is still stuff to be done
Mohit
@mvarshney1988
Mar 07 2017 05:01
for search, are you using solr/elastic?
Patrick Artounian
@partounian
Mar 07 2017 05:01
Just to confirm is there pieces of code that don't do anything? It seems like they might have previously but with updates are now doing nothing?
Brent Hoover
@zenweasel
Mar 07 2017 05:01
I think the structure of the plugins are pretty self-explanatory, yes?
Patrick Artounian
@partounian
Mar 07 2017 05:01
Kind of yes, but at times I have changed things with no changes/no breaking so I thought asking wouldn't hurt.
Brent Hoover
@zenweasel
Mar 07 2017 05:02
@mvarshney1988 No. We just used Mongo’s built-in text search. Although you can always replace this with your own
but Mongo yields sub 10-ms reponse times
Mohit
@mvarshney1988
Mar 07 2017 05:03
ohk , but data size matters here like 20-30 million products
?
Brent Hoover
@zenweasel
Mar 07 2017 05:03
@mvarshney1988 Yes, then you would need to use something like Elastic
Patrick Artounian
@partounian
Mar 07 2017 05:03
Probably would look at third party then.
Brent Hoover
@zenweasel
Mar 07 2017 05:03
are people still using Solr?
Mohit
@mvarshney1988
Mar 07 2017 05:03
i saw people using elastic
any distributing caching we are using here?
Brent Hoover
@zenweasel
Mar 07 2017 05:04
the front-end and backend are decoupled, so any module that provides the ProductSearch publication can be used by the front-end
@mvarshney1988 No
Mohit
@mvarshney1988
Mar 07 2017 05:05
ohk
Brent Hoover
@zenweasel
Mar 07 2017 05:05
with 20-30 million products, pretty much nothing is going to work for you out of the box
@partounian Yes, it’s very possible that there are pieces of code that are no longer used.
That’s a hard thing to keep an eye on, and just hasn’t been a priority compared to adding basic features
Mohit
@mvarshney1988
Mar 07 2017 05:07
depends on large scaling and distributed architecture
Patrick Artounian
@partounian
Mar 07 2017 05:08
If I come across things that seem to do nothing would it be worth it to collect them and put in a PR?
Brent Hoover
@zenweasel
Mar 07 2017 05:08
@mvarshney1988 Right. To my point.
@partounian Yes, especially if they are in the client or server directories from the root
Mohit
@mvarshney1988
Mar 07 2017 05:09
do you have any comparison stats with other ecommerce?
Brent Hoover
@zenweasel
Mar 07 2017 05:09
Even more especially if they are in the client
@mvarshney1988 I don’t. I can put you in touch with someone in sales if you want
Pretty much all we have in terms of top-line feature stuff is at http://reactioncommerce.com
It should be getting an update in a week or two with more/better information
Mohit
@mvarshney1988
Mar 07 2017 05:11
any demo website ?
Brent Hoover
@zenweasel
Mar 07 2017 05:12
No, we usually just encourage people to try it locally
that’s what most people want to do anyway
Mohit
@mvarshney1988
Mar 07 2017 05:13
yeah , i installed locally , but seems like we need to contribute in UI
Patrick Artounian
@partounian
Mar 07 2017 05:13
It is still in the early stages and seems that one of the upcoming milestones in UI/UX work.
Without functionality it doesn't matter how pretty it is :)
Brent Hoover
@zenweasel
Mar 07 2017 05:14
this current release is all cleanup/bug-fixes preparing for our 1.0 release
Mohit
@mvarshney1988
Mar 07 2017 05:15
@partounian correct :)
Brent Hoover
@zenweasel
Mar 07 2017 05:15
v0.20.0 should be coming out in the next few days
Patrick Artounian
@partounian
Mar 07 2017 05:15
Oops my mistake, at first looking at Github projects I just saw UIX with v0.20 so assumed it would be part of it.
Brent Hoover
@zenweasel
Mar 07 2017 05:16
both this last release and this one are UI/UX focused, but it’s been more about the admin and cleanup
we don’t expect that almost anyone will be using the UI “stock"
but we are moving towards being able to change the UI via substitution of React components for ours
Mohit
@mvarshney1988
Mar 07 2017 05:18
any document for setting up this as on android?
Brent Hoover
@zenweasel
Mar 07 2017 05:19
@mvarshney1988 No, that’s not something we actively support right now, though it is on the roadmap
Theoretically it should just build…..
Patrick Artounian
@partounian
Mar 07 2017 05:19
This might be a dumb question, but is there a smart way of overriding JS functions? For example, I don't want to show product range at all, either current price or the min. For one part I overwrote the displayPrice as we discussed yesterday but can't see a smart way to get that done on the productDetail page (with variants)
Brent Hoover
@zenweasel
Mar 07 2017 05:20
As I mentioned, in v20 we are introducing a way to override elements of the PDP via registering a React component and replacing ours
let me see if I can find the docs for that
Patrick Artounian
@partounian
Mar 07 2017 05:21
Apologies, I didn't realize it meant everything.
Brent Hoover
@zenweasel
Mar 07 2017 05:21
It’s just the Product Detail Page for now
that code should be in the branch wip-release-0.20.0
this is a branch new feature so I am still trying to wrap my head around it as well
Patrick Artounian
@partounian
Mar 07 2017 05:25
Did you mean brand new, or the branch in general has many new features?
Brent Hoover
@zenweasel
Mar 07 2017 05:26
sorry, meant to say “brand new"
so the ultimate goal here is that you could author a plugin that can override the product detail page without having to touch core code
Patrick Artounian
@partounian
Mar 07 2017 05:27
So it seems it would make more sense to work off of that branch instead of master, correct?
Brent Hoover
@zenweasel
Mar 07 2017 05:27
yeah, if you don’t mind a little breakage.
We obviously test stuff that goes into mmaster a little more vigourously
but if you aren’t super worried about stability, then yes
especially if you want this new feature
Patrick Artounian
@partounian
Mar 07 2017 05:30
Anything that needs to be run besides checking out the branch?
reaction reset for example?
Brent Hoover
@zenweasel
Mar 07 2017 05:30
You might want to run meteor npm install
Yeah, reaction reset is probably always a good idea
Patrick Artounian
@partounian
Mar 07 2017 05:32
What does the -n flag do for reset? I don't see it on help or a quick glance on the repo?
Brent Hoover
@zenweasel
Mar 07 2017 05:33
It bypasses the question about whether you want to reset all your node modules
I have an alias for rrr to reaction reset -n && reaction
Patrick Artounian
@partounian
Mar 07 2017 05:38
Sorry, but what is PDP?
ProductDetailPage?
Brent Hoover
@zenweasel
Mar 07 2017 05:40
Yes, sorry. The product detail page. Where all the magic happens
Patrick Artounian
@partounian
Mar 07 2017 05:57
I know you mentioned this the doc page, but that is only for react overwriting correct? What if I want to overwrite a function like getVariantPriceRange in lib/api/catalog.js?
Brent Hoover
@zenweasel
Mar 07 2017 06:11
why would you want to overwrite that function?
Couldn’t you just write your own components that call your own function?
because you aren’t going to affect just the display but also the admin
Patrick Artounian
@partounian
Mar 07 2017 06:21
Well some small things are only showing min, as I would not be using range, and also turning the variant's into price modifiers instead of replacements. So playing around.
(The second part probably involves another function, but you get the point)
Brent Hoover
@zenweasel
Mar 07 2017 06:22
I see
Alexander Berger
@bergsalex
Mar 07 2017 21:38
I'm wondering if anyone could point me in the right direction to overriding the default color on the admin dashboard. I'm making a theme, and can't stand leaving it as white. It also seems like the color is coded into the template.
<div class="rmq-459cc65c rui admin action-view-pane action-view open" data-radium="true" style="flex-direction: row; height: 100vh; position: relative; width: 400px; min-width: 400px; flex: 0 0 auto; background-color: white; overflow: hidden; z-index: 1050; transform: translateX(0px);">
Looking specifically at the background-color: white; within the style= section.
Mateo Morrison
@mateomorrison
Mar 07 2017 21:51
Hey! I asked something on github and they sent me here: reactioncommerce/reaction#1945
I deployed with Docker my reaction app. Added on docker-compse.yml my Reaction USER, MAIL and AUTH. When trying to access my app with the email I entered, it says it exist but I can't confirm it. How tf can I confirm my email?
tdmoneybanks
@tdmoneybanks
Mar 07 2017 21:53
@mateomorrison did you set up a mail url?
Mateo Morrison
@mateomorrison
Mar 07 2017 21:53
@tdmoneybanks No, how do I link Mailgun with it?
tdmoneybanks
@tdmoneybanks
Mar 07 2017 21:54
your not getting an email verification because the app isnt equiped to send out email
if you looked in the server logs
youd see an error regarding this
hold on a moment and let me see where you set it
Mateo Morrison
@mateomorrison
Mar 07 2017 21:55
tdmoneybanks
@tdmoneybanks
Mar 07 2017 21:55
please see here
the docs are slightly out of date as the dashboard is now under your user dropdown
but the steps to edit the settings should be the same
Spencer Norman
@spencern
Mar 07 2017 21:56
note that you won’t be able to edit that without logging in. But you can also setup MAIL_URL as an env variable on your server I think
Mateo Morrison
@mateomorrison
Mar 07 2017 21:56
Is there any file in the repo that I can modify? Since I can't access any admin dashboard and the advanced are too advanced for me
tdmoneybanks
@tdmoneybanks
Mar 07 2017 21:56
yes
the meteor settings file
look at the bottom of the link i just sent
and it tells you how to edit directly in code base
Mateo Morrison
@mateomorrison
Mar 07 2017 21:57
(Sorry newbie) where is settings.json? Remember I am using Docker to deploy it
Spencer Norman
@spencern
Mar 07 2017 21:57
are you deploying the from the reaction docker repo?
Mateo Morrison
@mateomorrison
Mar 07 2017 21:57
Yes
tdmoneybanks
@tdmoneybanks
Mar 07 2017 21:57
absolute simplest way
set the MAIL_URL env var in this file

this fiel
**
sorry edit the env var is docker-compose not dockerfile
Mateo Morrison
@mateomorrison
Mar 07 2017 21:59
I did set up reaction mail, user and auth in the docker-compose.yml
Spencer Norman
@spencern
Mar 07 2017 21:59
are you deploying like this:
docker run -d \
  -p 80:3000 \
  -e ROOT_URL="http://<your app url>" \
  -e MONGO_URL="mongodb://<your mongo url>" \
  -e REACTION_EMAIL="youradmin@yourdomain.com" \
  -e REACTION_USER="admin-username" \
  -e REACTION_AUTH="admin-password" \
  youraccount/reaction:latest
Mateo Morrison
@mateomorrison
Mar 07 2017 22:00
Nope, like that it doesn't do anything. I deployed like docker-compose up -d
tdmoneybanks
@tdmoneybanks
Mar 07 2017 22:00
@mateomorrison you said you set REACTION_EMAIL ?
Spencer Norman
@spencern
Mar 07 2017 22:00
i think you can still use -e
to set env variables
tdmoneybanks
@tdmoneybanks
Mar 07 2017 22:00
@mateomorrison ??
Spencer Norman
@spencern
Mar 07 2017 22:00
REACTION_EMAIL is your shop email address (from)
tdmoneybanks
@tdmoneybanks
Mar 07 2017 22:01
what email env var did you set?
what was it called
might be best to just print out your yaml file here
Spencer Norman
@spencern
Mar 07 2017 22:01
you need to setup your MAIL_URL as your smtp connection string
Mateo Morrison
@mateomorrison
Mar 07 2017 22:02

``` ##docker-compose.yml file contents
reaction:
image: reactioncommerce/reaction
links:

  - mongo
ports:
  - "80:80"
environment:
  ROOT_URL: "http://localhost"
  MONGO_URL: "mongodb://mongo:27017/meteor"
  REACTION_EMAIL: "john.kennedy@acquia.com"
  REACTION_USER: "admin"
  REACTION_AUTH: "password"

mongo:
image: mongo:latest
ports:

  - "27017:27017"
command: mongod --storageEngine=wiredTiger

mongo-express:
image: mongo-express:latest
ports:

  - "8081:8081"
links:
  - mongo``` 
tdmoneybanks
@tdmoneybanks
Mar 07 2017 22:02
yep
your mising the env var called
"MAIL_URL"
add this env var and set the value to your mailgun connection string
and you should be in biz
Mateo Morrison
@mateomorrison
Mar 07 2017 22:03
So it should be like:
Spencer Norman
@spencern
Mar 07 2017 22:03
Should look something like this
MAIL_URL=smtp://MAILGUNUSER:PASSWORD@smtp.mailgun.org:587'
Mateo Morrison
@mateomorrison
Mar 07 2017 22:03
Perfect
in the docker-compose right?
Spencer Norman
@spencern
Mar 07 2017 22:03
yeah, just add a line below REACTION_AUTH
needs to be indented identically
Also, if you’re building for testing / developing, all of this might be overkill as it’s probably easier to use the reaction-cli
Mateo Morrison
@mateomorrison
Mar 07 2017 22:06
Yup! Tested already. This is for production (do you call going live like that right)
Spencer Norman
@spencern
Mar 07 2017 22:06
yeah :+1:
Mateo Morrison
@mateomorrison
Mar 07 2017 22:07
Sorry, last question. MAILGUNUSER is my e-mail right?
Spencer Norman
@spencern
Mar 07 2017 22:08
not typically
tdmoneybanks
@tdmoneybanks
Mar 07 2017 22:08
mailgun should be giving you the correct connection string
Spencer Norman
@spencern
Mar 07 2017 22:08
I think they assign a smtp username that is unique and random
tdmoneybanks
@tdmoneybanks
Mar 07 2017 22:08
most companies already generate the url you need for u
look in mailguns settings/dashboard
Also, if you’re using a free sandbox mailgun account, you’ll have to add your email that you want to send mail to to the list of authorized emails
Patrick Artounian
@partounian
Mar 07 2017 22:16
I noticed at random times (when not even using reaction) the CPU usage spikes to 90+%
Victor
@Tynnee
Mar 07 2017 22:16
hi everyone, do i need an invite token to be able to run locally? if yes someone pls send me an invite token
Patrick Artounian
@partounian
Mar 07 2017 22:16
I just had it on wip-release-0.20.0 trying master now
No token needed
Mateo Morrison
@mateomorrison
Mar 07 2017 22:16
@Tynnee No you don't need one
Patrick Artounian
@partounian
Mar 07 2017 22:17
Even during reaction run (last ouput is MongoDB) it is running at 100%+ CPU usage
Mateo Morrison
@mateomorrison
Mar 07 2017 22:18
What's your memory size? @partounian
Brent Hoover
@zenweasel
Mar 07 2017 22:18
if you are still building, it’s going to be at 100%
Patrick Artounian
@partounian
Mar 07 2017 22:18
10GB
Isn't that a little excessive?
Brent Hoover
@zenweasel
Mar 07 2017 22:19
once you hit App running at: http://localhost:3000/ it should return to normal
Patrick Artounian
@partounian
Mar 07 2017 22:19
For me it was running at ~90% at that time when it was running
Victor
@Tynnee
Mar 07 2017 22:20
running reaction run almost does nothing. gets stuck at started MongoDB and returns an error
Brent Hoover
@zenweasel
Mar 07 2017 22:21
@Tynnee What’s the error?
Victor
@Tynnee
Mar 07 2017 22:22

/Users/victorasante-kyei/.meteor/packages/coffeescript/.1.11.1_4.155didq++os+web.browser+web.cordova/plugin.compileCoffeescript.os/npm/node_modules/meteor/promise/node_modules/meteor-promise/promise_server.js:190
throw error;
^

Error: ENFILE: file table overflow, scandir '/Users/victorasante-kyei/Documents/React/reaction/packages/gridfs'
at Error (native)
at Object.fs.readdirSync (fs.js:808:18)
at Object.wrapper (/tools/fs/files.js:1535:35)
at readDirectory (/Users/victorasante-kyei/.meteor/packages/meteor-tool/.1.4.2_7.zqi29q++os.osx.x86_64+web.browser+web.cordova/mt-os.osx.x86_64/tools/fs/watch.js:291:26)
at Watcher._fireIfDirectoryChanged (/Users/victorasante-kyei/.meteor/packages/meteor-tool/.1.4.2_7.zqi29q++os.osx.x86_64+web.browser+web.cordova/mt-os.osx.x86_64/tools/fs/watch.js:436:23)
at /Users/victorasante-kyei/.meteor/packages/meteor-tool/.1.4.2_7.zqi29q++os.osx.x86_64+web.browser+web.cordova/mt-os.osx.x8664/tools/fs/watch.js:682:12
at Array.forEach (native)
at Function.
.each._.forEach (/Users/victorasante-kyei/.meteor/packages/meteor-tool/.1.4.2_7.zqi29q++os.osx.x86_64+web.browser+web.cordova/mt-os.osx.x86_64/dev_bundle/lib/node_modules/underscore/underscore.js:79:11)
at Watcher._checkDirectories (/Users/victorasante-kyei/.meteor/packages/meteor-tool/.1.4.2_7.zqi29q++os.osx.x86_64+web.browser+web.cordova/mt-os.osx.x86_64/tools/fs/watch.js:671:7)
at new Watcher (/Users/victorasante-kyei/.meteor/packages/meteor-tool/.1.4.2_7.zqi29q++os.osx.x86_64+web.browser+web.cordova/mt-os.osx.x86_64/tools/fs/watch.js:385:10)
at setupClientWatcher (/tools/runners/run-app.js:786:23)
at AppRunner._runOnce (/tools/runners/run-app.js:797:7)
at AppRunner._fiber (/tools/runners/run-app.js:876:28)
at /tools/runners/run-app.js:403:12

Mikkal McNulty
@MikkalMc
Mar 07 2017 22:22
I also get high CPU usage while running reaction, and this is after building
Patrick Artounian
@partounian
Mar 07 2017 22:22
ENFILE error is on the getting started docs
run the commands for increasing open file limit
Brent Hoover
@zenweasel
Mar 07 2017 22:22
Yep
Mateo Morrison
@mateomorrison
Mar 07 2017 22:27
@spencern @tdmoneybanks I set up everything (I think right) but it keeps telling that i have to validate my email
Let me try recovering the password
Spencer Norman
@spencern
Mar 07 2017 22:28
yeah, try resetting pw
tdmoneybanks
@tdmoneybanks
Mar 07 2017 22:34
@mateomorrison did you get a verification email?
Victor
@Tynnee
Mar 07 2017 22:34
thanks @partounian found it
Patrick Artounian
@partounian
Mar 07 2017 22:34
No problem @Tynnee
Mateo Morrison
@mateomorrison
Mar 07 2017 22:38
Nope. Not working... Maybe set up MAIL_URL the wrong way :)
Spencer Norman
@spencern
Mar 07 2017 22:39
do you have access to server logs?
Patrick Artounian
@partounian
Mar 07 2017 22:39
I hope Meteor updates to a newer version of Node for hopefully less usage
Mateo Morrison
@mateomorrison
Mar 07 2017 22:40
``` ERROR Reaction: Email job failed    ``` 
Spencer Norman
@spencern
Mar 07 2017 22:40
seems like maybe the MAIL_URL didn’t take
do you have access to the db?
one other way to get around this for now is to just manually set the user to email validated = true
Mateo Morrison
@mateomorrison
Mar 07 2017 22:41
MAIL_URL: "smtp://blabla:blabla@stmp.mailgun.org:587"
Spencer Norman
@spencern
Mar 07 2017 22:41
in the database
should be smtp not stmp
Mateo Morrison
@mateomorrison
Mar 07 2017 22:42
yes it is smtp
How to access my MongoDB? It's hosted locally in another container
Spencer Norman
@spencern
Mar 07 2017 22:42
not sure exactly. You’d probably have to open a port for it and I doubt that’s done by default :/
Mateo Morrison
@mateomorrison
Mar 07 2017 22:44
So maybe I did it wrong setting up the username and password. Since I can't access my MongoDB
Spencer Norman
@spencern
Mar 07 2017 22:44
that’s where I’d lean.
I do remember the mailgun username:password being more complicated than it needed to be
Mateo Morrison
@mateomorrison
Mar 07 2017 22:45
Any alternative?
Spencer Norman
@spencern
Mar 07 2017 22:45
we use mailgun for dev and postmark (postmarkapp.com) for production
but you’ll still have an SMTP connection string either way
Mateo Morrison
@mateomorrison
Mar 07 2017 22:47
Signed up to Postmark already @spencern how do I set up MAIL_URL
Spencer Norman
@spencern
Mar 07 2017 22:49
same way - I think postmark makes you verify your domain (SPF and DKIM)
for postmark you generate an API token that you use as both the username and the password
e.g. smtp://TOKEN:TOKEN@smtp.postmarkapp.com:587
but they will reject any mail that you haven’t verified the email sender for and the domain for
and it’s transactional only (fine for RC) - so no mass marketing emails
Mateo Morrison
@mateomorrison
Mar 07 2017 23:12
Now it got the Postmark smtp. Now it says user not found :)
Mateo Morrison
@mateomorrison
Mar 07 2017 23:27
I verified everything (literally)
Mateo Morrison
@mateomorrison
Mar 07 2017 23:43
@spencern It shows me this error:
23:40:37.145Z ERROR Reaction: Email job failed Error: connect EHOSTUNREACH 147.75.195.176:5887 at Object.exports._errnoException (util.js:907:11) at exports._exceptionWithHostPort (util.js:930:20) at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1078:14)var code = "formatted";