These are chat archives for reactioncommerce/reaction

17th
Dec 2015
Brent Hoover
@zenweasel
Dec 17 2015 02:41
Anyone who was running into the issue with Paypal Express just sitting on the spinner, are you still encountering it and are you using foreign (non-USD) currencies?
F4is4l
@F4is4l
Dec 17 2015 02:52
I haven't seen any website which has reaction deployed and running as primary e commerce , is there any ?
newsiberian
@newsiberian
Dec 17 2015 04:19
@zenweasel, yesterday I was first time tried to connect Paypal Express with Reaction defaults and I was saw this endless spinner.
Brent Hoover
@zenweasel
Dec 17 2015 04:23
@newsiberian Thanks. I assume you are using non-USD currency as well?
newsiberian
@newsiberian
Dec 17 2015 04:24
no, I'm using default currency
Brent Hoover
@zenweasel
Dec 17 2015 04:25
Hmmm, that’s very odd
newsiberian
@newsiberian
Dec 17 2015 04:26
But maybe I was wrong with PayPal configuration. I was encountered with some issues in sandbox accounts registration. Maybe that is the cause.
Brent Hoover
@zenweasel
Dec 17 2015 04:28
Well if you run into again let me know. I have never run into it using the default currency, but when I switched currencies I did see it.
@newsiberian Everything on the new version and the Paypal package up to 1.2.5?
newsiberian
@newsiberian
Dec 17 2015 04:30
ofc
Brent Hoover
@zenweasel
Dec 17 2015 04:31
lol, sorry. By law I have to ask
newsiberian
@newsiberian
Dec 17 2015 04:31
:smile:
newsiberian
@newsiberian
Dec 17 2015 04:49

@aaronjudd, Hi, I am running into this:

W20151217-10:47:35.745(6)? (STDERR)           
W20151217-10:47:35.746(6)? (STDERR) /home/sohaam/.meteor/packages/meteor-tool/.1.1.10.gjfrq3++os.linux.x86_64+web.browser+web.cordova/mt-os.linux.x86_64/dev_bundle/server-lib/node_modules/fibers/future.js:245
W20151217-10:47:35.747(6)? (STDERR)                                             throw(ex);
W20151217-10:47:35.748(6)? (STDERR)                                                   ^
W20151217-10:47:35.748(6)? (STDERR) Error: Units of measure is required
W20151217-10:47:35.748(6)? (STDERR)     at getErrorObject (meteor://💻app/packages/aldeed_collection2/lib/collection2.js:331:1)
W20151217-10:47:35.748(6)? (STDERR)     at [object Object].doValidate (meteor://💻app/packages/aldeed_collection2/lib/collection2.js:314:1)
W20151217-10:47:35.748(6)? (STDERR)     at [object Object].Mongo.Collection.(anonymous function) [as insert] (meteor://💻app/packages/aldeed_collection2/lib/collection2.js:83:1)
W20151217-10:47:35.748(6)? (STDERR)     at Function.Factory._create (meteor://💻app/packages/dburles_factory/packages/dburles_factory.js:87:1)
W20151217-10:47:35.748(6)? (STDERR)     at Function.Factory.create (meteor://💻app/packages/dburles_factory/packages/dburles_factory.js:95:1)
W20151217-10:47:35.748(6)? (STDERR)     at Object.faker.reaction.shops.getShop (meteor://💻app/common/factories/faker.js:223:36)
W20151217-10:47:35.748(6)? (STDERR)     at Factory.define.shopId (meteor://💻app/common/factories/products.js:11:32)
W20151217-10:47:35.748(6)? (STDERR)     at /tmp/meteor-test-runzibc19/.meteor/local/build/programs/server/packages/reactioncommerce_reaction-factories.js:607:4
W20151217-10:47:35.749(6)? (STDERR)     at /tmp/meteor-test-runzibc19/.meteor/local/build/programs/server/packages/reactioncommerce_reaction-factories.js:727:3
W20151217-10:47:35.749(6)? (STDERR)     at /tmp/meteor-test-runzibc19/.meteor/local/build/programs/server/boot.js:242:10
W20151217-10:47:35.749(6)? (STDERR)     at Array.forEach (native)
W20151217-10:47:35.749(6)? (STDERR)     at Function._.each._.forEach (/home/sohaam/.meteor/packages/meteor-tool/.1.1.10.gjfrq3++os.linux.x86_64+web.browser+web.cordova/mt-os.linux.x86_64/dev_bundle/server-lib/node_modules/underscore/underscore.js:79:11)
W20151217-10:47:35.749(6)? (STDERR)     at /tmp/meteor-test-runzibc19/.meteor/local/build/programs/server/boot.js:137:5
=> Exited with code: 8

with this: VELOCITY_TEST_PACKAGES=1 meteor test-packages --driver-package velocity:html-reporter --port 3006 reactioncommerce:reaction-accounts

Aaron Judd
@aaronjudd
Dec 17 2015 07:51
@newsiberian pull the latest reaction-factories, (or meteor update, it's a published package)
pretty sure that’s from an outdated Factory, as Units of Measure should now be defined on Factory.create(“shop”)
(or, your Shops collection document is out of date)
newsiberian
@newsiberian
Dec 17 2015 08:00
It runs fine now with factories from github
Aaron Judd
@aaronjudd
Dec 17 2015 08:00
:thumbsup:
btw - I just clone into packages and once in a while reaction pull keeps things in sync ;-)
newsiberian
@newsiberian
Dec 17 2015 08:02
thanks :smile:
newsiberian
@newsiberian
Dec 17 2015 08:07
@zenweasel, i've tried paypal express again with new buyer mail and it works fine with USD, but not working with ruble. Maybe we send currency in wrong format to paypal?
@aaronjudd, returning to the question about ongo:security. Did you look at how it work with server-side operations?
Brent Hoover
@zenweasel
Dec 17 2015 08:22
@newsiberian Thanks. I am working on the currency issue right now.
Aaron Judd
@aaronjudd
Dec 17 2015 08:30
@newsiberian I haven’t yet - if you want to create an issue with steps to replicate what you’re seeing I’ll dig into it tomorrow
newsiberian
@newsiberian
Dec 17 2015 08:36
@aaronjudd then one small question: should this package work with operations called from server side methods?
Aaron Judd
@aaronjudd
Dec 17 2015 09:16
Bogi
@boboci9
Dec 17 2015 11:50

Hi, I found an issue on the product form -> variantForm.js:

childVariants: function () {
    let product = selectedProduct();
    if (!product) {
      return {};
    }
....
}

while on the temlate you have:

 {{#each childVariants}}
          {{>childVariantForm}}
        {{/each}}

and in case you don't have a selectedProduct() this error will be thrown:

Error: {{#each}} currently only accepts arrays, cursors or falsey values.

I think it should be return null; or return []; or anything else but an empty object

Aaron Judd
@aaronjudd
Dec 17 2015 16:52
:point_up: @boboci9 I created an issue for this: reactioncommerce/reaction#599
Israel Ortiz
@soyisraelortiz
Dec 17 2015 16:52
wow, the new sidebar is BEAUTIFUL!!!!!!!
Aaron Judd
@aaronjudd
Dec 17 2015 16:53
@nubestra :thumbsup: work in progress, work in progress ;-)
Bogi
@boboci9
Dec 17 2015 17:49
ok thanks @aaronjudd, I just didn't want to create a new issue if someone has already dealt with it
Aaron Judd
@aaronjudd
Dec 17 2015 19:18
@boboci9 no problems, we’re in a bug fix cycle - so the more that are reported the more we’ll push to resolve for 0.11 :thumbsup:
Bogi
@boboci9
Dec 17 2015 19:19
:+1:
Stefan Smiljkovic
@shtefcs
Dec 17 2015 21:41
anyone know how to setup admin cred for production ?
Aaron Judd
@aaronjudd
Dec 17 2015 22:43
@shtefcs in what way?
(you can pass in settings, or ENV)