Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

This one weird trick will fix your Shopify import problems #3111

Merged
merged 2 commits into from
Oct 17, 2017

Conversation

spencern
Copy link
Contributor

This seems to fix the Shopify importer for me.

Not sure why that would be the case, but putting this up here for now, I'm gonna pull it down and run it through on a fresh install in a bit.

@spencern
Copy link
Contributor Author

@mikemurray this fixes the problems with the importer.. but it doesn't make sense to me as why it's fixing it. There are issues with the importer running afoul of Meteor fibers when the package.json file is not present, but this doesn't really feel like a clean fix to me.
I'm gonna keep looking at it, but this might do to let us keep experimenting with the onboarding flow for now.

@spencern spencern changed the title This one weird trick will fix your Shopify import problems [WIP] This one weird trick will fix your Shopify import problems Oct 16, 2017
@spencern
Copy link
Contributor Author

Adding the WIP flag here because I want to re-write the way this works so that it's not dependent on whatever is happening with the package.json - it should work in either scenario.

@brent-hoover brent-hoover self-requested a review October 16, 2017 23:59
@brent-hoover brent-hoover changed the title [WIP] This one weird trick will fix your Shopify import problems This one weird trick will fix your Shopify import problems Oct 17, 2017
@brent-hoover brent-hoover merged commit 0c5133d into release-1.5.1 Oct 17, 2017
@brent-hoover brent-hoover deleted the spencer-fix-shopify-connector branch October 17, 2017 00:01
@mulianto mulianto mentioned this pull request Oct 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants