These are chat archives for reactioncommerce/reaction

17th
Aug 2016
Owen Hoskins
@owenhoskins
Aug 17 2016 09:17
Hey all, got a Q regarding default data imports, I am seeing a bunch of output on start up like this:
I20160817-11:00:17.431(2)? 09:00:17.241Z  WARN Reaction: Error while importing to Products: After applying the update to the document {_id: "vcZzAeXaQGnYD8ghR" , ...}, the (immutable) field '_id' was found to have been altered to _id: "yiMQkkWrtowvzjogo" , ...}, the (immutable) field '_id' was found to have been altered to _id: "yiMQkkWrtowvzjogo"
It seems to prevent one products variants to be imported. Seen this kind of thing before?
RefactorMan
@RefactorMan
Aug 17 2016 09:23
@aaronjudd thanks for your help
Brent Hoover
@zenweasel
Aug 17 2016 10:30
@owenhoskins I know this isn’t super helpful, but I have seen that same error and have had no luck in tracking it down yet. It seems to have started with the change to Meteor 1.4 but I am not 100% sure
Owen Hoskins
@owenhoskins
Aug 17 2016 10:31
Ok, g2k, I will let you know if I have any luck!
@zenweasel: This might be totally off, but I think in my case it has something to do with products sharing the same "title" or perhaps handle at one point.
Brent Hoover
@zenweasel
Aug 17 2016 10:42
Hmmm. It happened to me with exporting and then reimporting products for the example-plugin
Owen Hoskins
@owenhoskins
Aug 17 2016 10:43
That's definitely part of it for me as well. I think I may have another issue that is confusing it even more.
Owen Hoskins
@owenhoskins
Aug 17 2016 11:01
@zenweasel I may of found my issue: /server/publications/collections/product.js line 43. Looks to be selecting the first doc with that handle.
Actually now that I logged the result of products = Products.find(selector).fetch(); I am seeing a whole array of products who all have "chair" in their titles
Brent Hoover
@zenweasel
Aug 17 2016 11:08
not sure I understand how a publication affects an issue with importing
Owen Hoskins
@owenhoskins
Aug 17 2016 11:09
Sorry -- that is unrelated to importing.
Brent Hoover
@zenweasel
Aug 17 2016 11:09
Ah ok
Owen Hoskins
@owenhoskins
Aug 17 2016 11:10
I am having a look at the mongo $regex selector. The current implementation finds all products that share any part of a product title.
Brent Hoover
@zenweasel
Aug 17 2016 11:16
yeah, I am not sure I understand the use of $regex there
Owen Hoskins
@owenhoskins
Aug 17 2016 11:21
I think I'll just enforce unique handles as much as possible for the time being. But in general do you see this as an issue?
Brent Hoover
@zenweasel
Aug 17 2016 11:22
if you had a “chair” and then a “red chair” and you navigated to “chair” you would get both (in the find) at least if I am understanding it correctly
Owen Hoskins
@owenhoskins
Aug 17 2016 11:22
yeah that is what I am seeing
Brent Hoover
@zenweasel
Aug 17 2016 11:23
that’s can’t be right
I mean, I don’t think that’s what was intended, but it does look like what was implemented, but would need to test to know for sure
Owen Hoskins
@owenhoskins
Aug 17 2016 11:24
agreed!
Brent Hoover
@zenweasel
Aug 17 2016 11:25
there must be some reasoning behind the $regex otherwise why wouldn’t you just use an regular match
ishaq-bhojani
@ishaq-bhojani
Aug 17 2016 15:20
Hi there!

I'm having an issue while installing reaction
I'm new with meteor

reaction@0.14.2 postinstall C:\Users\Ishaq Bhojani\Desktop\Reaction\another1
.reaction/scripts/postinstall.sh

'.reaction' is not recognized as an internal or external command,
operable program or batch file.

npm ERR! Windows_NT 10.0.10586

Aaron Judd
@aaronjudd
Aug 17 2016 15:57
@ishaq-bhojani I think this is fixed in the development branch (v0.15.0), which we should be releasing today.
ishaq-bhojani
@ishaq-bhojani
Aug 17 2016 15:59
@aaronjudd is it the problem with reaction-cli
Aaron Judd
@aaronjudd
Aug 17 2016 16:01
@ishaq-bhojani I think this is #1299 which removed the post install from npm package.json
bash scripts don’t play nicely on windows
ishaq-bhojani
@ishaq-bhojani
Aug 17 2016 16:03
so what should i do now?
to run the project
Aaron Judd
@aaronjudd
Aug 17 2016 16:04
try reaction init -b development
that will use the development branch, if you just want to check it out
ishaq-bhojani
@ishaq-bhojani
Aug 17 2016 16:06
thanks @aaronjudd
ishaq-bhojani
@ishaq-bhojani
Aug 17 2016 16:27

@aaronjudd Im getting these logs
=> Errors prevented startup:

While building for web.browser:
imports/plugins/included/default-theme/client/styles/main.less:6: Unknown import:
{}/node_modules/bootstrap/less/variables.less

While minifying app stylesheet:
error: Can't load npm module 'autoprefixer' from packages/minifier-postcss_plugin.js. Check your Npm.depends().

=> Your application has errors. Waiting for file change.

tried npm install and metor npm install
but it stops with these errors
npm ERR! Windows_NT 10.0.10586
npm ERR! argv "C:\Users\Ishaq Bhojani\AppData\Local\.meteor\packages\meteor-tool\1.4.0-1\mt-os.windows.x86_32\dev_bundle\bin\node.exe" "C:\Users\Ishaq Bhojani\AppData\Local\.meteor\packages\meteor-tool\1.4.0-1\mt-os.windows.x86_32\dev_bundle\lib\node_modules\npm\bin\npm-cli.js" "install"
npm ERR! node v4.4.7
npm ERR! npm v3.10.5
npm ERR! code 128
Aaron Judd
@aaronjudd
Aug 17 2016 16:28
reaction reset should take care of that I think.
or as the error says “meteor npm install”, looks like not all your node_modules got installed
ishaq-bhojani
@ishaq-bhojani
Aug 17 2016 18:09
@aaronjudd
got this error on reaction reset
npm ERR! git clone --template
Brent Hoover
@zenweasel
Aug 17 2016 23:36
@owenhoskins Just doing some testing this morning and it would appear that that publication works correctly. I get 2 products (the top level and the variant) for both “chair” and “red chair"