These are chat archives for reactioncommerce/reaction

16th
Mar 2015
Aaron Judd
@aaronjudd
Mar 16 2015 01:40
@ptotaram are you using the development branch?
pull everything fresh also, just merged the guest checkout branch with a lot of fixes, so you’ll want to make sure that you have the latest (as well as pull the development packages) that are listed in .meteor/packages or remove some of them if you don’t want to pull them all down
Goutham Veeramachaneni
@gouthamve
Mar 16 2015 05:09
Hi I am getting an error after I pulled from development
   /Users/goutham/.meteor/packages/meteor-tool/.1.0.41.azxhr5++os.osx.x86_64+web.browser+web.cordova/meteor-tool-os.osx.x86_64/dev_bundle/lib/node_modules/fibers/future.js:323:16:
   Unknown asset: theme/dashboard/shop/accounts.less (compiling
   client/themes/custom.reaction.json)
   at Object.Future.wait
   (/Users/goutham/.meteor/packages/meteor-tool/.1.0.41.azxhr5++os.osx.x86_64+web.browser+web.cordova/meteor-tool-os.osx.x86_64/dev_bundle/lib/node_modules/fibers/future.js:323:16)
   at getAsset
   (/Users/goutham/.meteor/packages/meteor-tool/.1.0.41.azxhr5++os.osx.x86_64+web.browser+web.cordova/meteor-tool-os.osx.x86_64/tools/bundler.js:1298:20)
   at Object._.extend.Assets.getText
   (/Users/goutham/.meteor/packages/meteor-tool/.1.0.41.azxhr5++os.osx.x86_64+web.browser+web.cordova/meteor-tool-os.osx.x86_64/tools/bundler.js:1352:20)
   at ThemeData (<runJavaScript-38>:18:17)
   at getAsset (<runJavaScript-39>:133:10)
   at getLessContent (<runJavaScript-39>:137:17)
   at <runJavaScript-39>:257:27
   at Function._.each._.forEach (<runJavaScript-36>:147:22)
   at handler (<runJavaScript-39>:256:5)
Aaron Judd
@aaronjudd
Mar 16 2015 05:12
I’m not sure if you also have the updated https://github.com/reactioncommerce/reaction-core-theme, if not you’ll need to clone that into packages
Goutham Veeramachaneni
@gouthamve
Mar 16 2015 05:12
Oh. I did not. I will do that.
Aaron Judd
@aaronjudd
Mar 16 2015 05:14
I’m guessing that’s it.. things start breaking with the published packages, it’s usually safer to use & pull the package repos down into packages. I need a better release process before I can start publishing “dev” packages :worried:
Goutham Veeramachaneni
@gouthamve
Mar 16 2015 05:15
Hmm. That did not solve it.
Changes to your project's package version selections:

reactioncommerce:core-theme  upgraded from 1.2.2 to 1.3.0

=> Errors prevented startup:                  

   While building the application:

   /Users/goutham/.meteor/packages/meteor-tool/.1.0.41.azxhr5++os.osx.x86_64+web.browser+web.cordova/meteor-tool-os.osx.x86_64/dev_bundle/lib/node_modules/fibers/future.js:323:16:
   Unknown asset: theme/dashboard/shop/accounts.less (compiling
   client/themes/custom.reaction.json)
   at Object.Future.wait
I cloned but still there are problems.
Aaron Judd
@aaronjudd
Mar 16 2015 05:17
ah, 2 seconds.. looks like I forgot to merge the guest checkout branch over into development (on the theme)
@Gouthamve ok pull the theme development branch
Goutham Veeramachaneni
@gouthamve
Mar 16 2015 05:22
done. :+1:
Checkout page looks good.
Aaron Judd
@aaronjudd
Mar 16 2015 05:24
:thumbsup:
Goutham Veeramachaneni
@gouthamve
Mar 16 2015 05:35
Minor bug. It is showing Your order is now cartCompleted.submitted
Screen Shot 2015-03-16 at 11.04.21 am.png
Aaron Judd
@aaronjudd
Mar 16 2015 05:58
you might need to reload your Translation files..
Goutham Veeramachaneni
@gouthamve
Mar 16 2015 05:58
Also #16 is not yet resolved.
And how do I reload the Translation files?
Aaron Judd
@aaronjudd
Mar 16 2015 06:01
I usually remove all Translations documents from the db, and restart meteor
but I actually wrote a client method for that…. probably documented in the i18n doc
Goutham Veeramachaneni
@gouthamve
Mar 16 2015 06:02
Oh. I will check it out.
Aaron Judd
@aaronjudd
Mar 16 2015 06:03
well, I don’t see it in the docs.. anyways..it’s easy to do
(or meteor reset and lose your test data)
probably a good idea to start with fresh data anyways
Goutham Veeramachaneni
@gouthamve
Mar 16 2015 06:07
And #16 was resolved when I meteor reset
Goutham Veeramachaneni
@gouthamve
Mar 16 2015 06:18
Also now If I am logged in, I cannot add anything to cart!
Aaron Judd
@aaronjudd
Mar 16 2015 06:19
I see that - i’m looking into that right now - something new, or something that didn’t get committed - not sure
Aaron Judd
@aaronjudd
Mar 16 2015 06:47
@Gouthamve I think you can pull now and that should be fixed. if not can you create an issue with steps to replicate. btw - start with a nice clean session, and meteor reset, if possible
I’m done for today.. but I’ll grab any bugs for this release right away
Pream Totaram
@ptotaram
Mar 16 2015 11:09
@aaronjudd I am using that branch, thanks I will do
Goutham Veeramachaneni
@gouthamve
Mar 16 2015 12:21
Hmm. So I looked into Indian payment gateways. All of them are like, we will redirect the customer to the gateway providers website, all txns are done there and the person is redirected back to our website.
Goutham Veeramachaneni
@gouthamve
Mar 16 2015 12:29
@aaronjudd Is there anyway that Reaction supports this setup currently, or should I wait before I develop the gateways for Reaction.
I am thinking of developing integrations for the top two payment gateway providers, PayU and CCavenue.
Goutham Veeramachaneni
@gouthamve
Mar 16 2015 14:28
And is there any scope for <iframe> integrations for payment?
Aaron Judd
@aaronjudd
Mar 16 2015 14:32
@gouthamve the callbacks are probably better than an iframe payment integration.. you’ll probably just need to add a iron:router route to the package that handles the payment providers webhook/callback
Goutham Veeramachaneni
@gouthamve
Mar 16 2015 14:34
Oh. Okay.
Aaron Judd
@aaronjudd
Mar 16 2015 14:37
there’s nothing stopping you from putting an iframe in there (but, ugh)
Pream Totaram
@ptotaram
Mar 16 2015 15:52
lol, iframes are something you gotta deal with, I've had to deal with that with a client that used chase
Bogi
@boboci9
Mar 16 2015 15:55
Hi, in the new version of reaction core did anyone fount this issue? Cannot read property 'length' of undefined
at Object.loadI18n ?
Aaron Judd
@aaronjudd
Mar 16 2015 15:56
haven’t seen that -> do you have a way to replicate? is it on every page?
(have you reset / updated Translations? - if not, there shouldn’t be an error, but it’s a hint to me)
Bogi
@boboci9
Mar 16 2015 15:57
Yes I reset the app, it is since I reset the DB, and yes it appears on every page
Aaron Judd
@aaronjudd
Mar 16 2015 16:03
does shop.languages exist in your shops collection?
@boboci9 that’s a server console error right?
Bogi
@boboci9
Mar 16 2015 16:06
yes it appears on the server
and from what I see not all the fixtures were created correctly
Aaron Judd
@aaronjudd
Mar 16 2015 16:07
it sounds like Shops didn’t get created properly
Bogi
@boboci9
Mar 16 2015 16:07
TypeError: Cannot read property 'length' of undefined
at Object.loadI18n (/home/bogi/Meteor/newco/reaction/.meteor/local/build/programs/server/packages/reactioncommerce_core.js:3441:31)
at loadFixtures (/home/bogi/Meteor/newco/reaction/.meteor/local/build/programs/server/packages/reactioncommerce_core.js:3522:12)
at __coffeescriptShare (/home/bogi/Meteor/newco/reaction/.meteor/local/build/programs/server/packages/reactioncommerce_core.js:3574:3)
at /home/bogi/Meteor/newco/reaction/.meteor/local/build/programs/server/boot.js:212:5
so the error comes from loading the Fixtures
I will try again, but this time I will take out my own fixtures, if noone else has this issue, than it could come from my fixtures too :)
Aaron Judd
@aaronjudd
Mar 16 2015 16:09
I’d take a look at Shops… I’m going to put in handling for that error though
Bogi
@boboci9
Mar 16 2015 16:11
yes the shop was not created because my fixtures also had a shop, sorry I fixed it by taking out my fixtures
Aaron Judd
@aaronjudd
Mar 16 2015 16:13
cool, thanks
Pream Totaram
@ptotaram
Mar 16 2015 16:45
after the last pull I can't build the application any more
I20150316-16:41:58.017(0)? => GraphicsMagick found
I20150316-16:41:58.390(0)? Meteor._wrapAsync has been renamed to Meteor.wrapAsync
W20150316-16:41:58.692(0)? (STDERR)
W20150316-16:41:58.692(0)? (STDERR) /home/vagrant/.meteor/packages/meteor-tool/.1.0.41.yumiah++os.linux.x86_64+web.browser+web.cordova/meteor-tool-os.linux.x86_64/dev_bundle/server-lib/node_modules/fibers/future.js:173
W20150316-16:41:58.692(0)? (STDERR) throw(ex);
W20150316-16:41:58.692(0)? (STDERR) ^
W20150316-16:41:58.697(0)? (STDERR) ReferenceError: BrowserPolicy is not defined
W20150316-16:41:58.698(0)? (STDERR) at server/policy.coffee:1:1
W20150316-16:41:58.698(0)? (STDERR) at server/policy.coffee:1:1
W20150316-16:41:58.698(0)? (STDERR) at /var/www/.meteor/local/build/programs/server/boot.js:205:10
W20150316-16:41:58.698(0)? (STDERR) at Array.forEach (native)
W20150316-16:41:58.698(0)? (STDERR) at Function..each..forEach (/home/vagrant/.meteor/packages/meteor-tool/.1.0.41.yumiah++os.linux.x86_64+web.browser+web.cordova/meteor-tool-os.linux.x86_64/dev_bundle/server-lib/node_modules/underscore/underscore.js:79:11)
W20150316-16:41:58.698(0)? (STDERR) at /var/www/.meteor/local/build/programs/server/boot.js:116:5
=> Exited with code: 8
=> Your application is crashing. Waiting for file change.
that's the error
Everest Liu
@evliu
Mar 16 2015 17:41
didn’t get the error; did you git pull reaction-core inside your packages folder as well?
Pream Totaram
@ptotaram
Mar 16 2015 18:23
no, thanks
Goutham Veeramachaneni
@gouthamve
Mar 16 2015 18:26
Also clone reaction-core-theme
Goutham Veeramachaneni
@gouthamve
Mar 16 2015 19:05
This message was deleted
This message was deleted
Goutham Veeramachaneni
@gouthamve
Mar 16 2015 20:09
@aaronjudd This might help with #347 https://youtu.be/Zu4LhVdGCn8
Aaron Judd
@aaronjudd
Mar 16 2015 22:47
@Gouthamve that confirmed what I was thinking.
Good screen work!