Following Factory import doesn't work
Closed this issue · 8 comments
Background: meteor/todos#119
Our testing article does:
import { Factory } from 'meteor/factory';
Which gives this error on the client:
install.js:78Uncaught Error: Cannot find module 'meteor/factory'
Which makes sense because factory is not a package on atmo. However, this:
import { Factory } from 'meteor/dburles:factory';
Also doesn't work:
install.js:78Uncaught Error: Cannot find module 'meteor/dburles:factory'
Isn't Factory a local Atmosphere package? You may need to meteor add factory?
It is in todos, but what about for people just following the guide snippets? I would change them to 'meteor/dburles:factory';, but that doesn't seem to work
On Saturday, May 7, 2016, Tom Coleman notifications@github.com wrote:
Isn't Factory a local Atmosphere package? You may need to meteor add
factory?—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#427 (comment)
Yeah, not ideal. I guess we need to update todos to use dburles factory
evebtually
On Sat, 7 May 2016 at 4:16 PM, Loren Sands-Ramshaw notifications@github.com
wrote:
It is in todos, but what about for people just following the guide
snippets?On Saturday, May 7, 2016, Tom Coleman notifications@github.com wrote:
Isn't Factory a local Atmosphere package? You may need to meteor add
factory?—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#427 (comment)—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#427 (comment)
Why does https://github.com/meteor/todos use its own factory implementation?
I'm copy pasting from the todos example to my own newly started project. The meteor/factory import confuses me.
For anyone still interested the todos example app now uses dburles:factory
I'm closing this issue because it's too old.
We are going to make many updates to the guide in the next weeks and it's better to focus on issues with recent activity. If you think this issue is still relevant please open a new one.