Originally created by: birkskyum
This pull request enable the new npm integration that came with Meteor 1.3. It isn't trivial because the packages that used to be loaded by meteor now needs an extra 'require()' - touching almost all files in the repo. Fixes [#37]
Activate "ecmascript" package to enable npm
Remove meteorhacks:npm and npm-container and use the new npm integration instead.
Load the following packages directly from npm:
This pr takes another, more careful, approach to achieving the same as [#138] without installing any 3rd-party atmosphere packages. I have checked that any of the visual errors found in that pull request doesn't exist here.
@rsercano Could you check that everything looks like it's supposed to?
View and moderate all "tickets Discussion" comments posted by this user
Mark all as spam, and block user from posting to "Tickets"
Originally posted by: rsercano
Hi Birk,
unfortunately I couldn't make that work, I just typed npm install before meteor command, am I doing something wrong ?
Here are the logs:
View and moderate all "tickets Discussion" comments posted by this user
Mark all as spam, and block user from posting to "Tickets"
Originally posted by: birkskyum
Try to run
meteor npm installinstead. The version of npm bundled with meteor might be different from the one you have installed.You can compare:
npm -vandmeteor npm -vView and moderate all "tickets Discussion" comments posted by this user
Mark all as spam, and block user from posting to "Tickets"
Originally posted by: rsercano
Oh, sorry... Tried it and now throwing chimp dependency errors such as: (this can be about windows distribution, don't know really)
Now I'm trying to remove chimp dependency and test, but chimp is necessary for travis build (not mandatory can find another workaround, since there're no tests yet.)
View and moderate all "tickets Discussion" comments posted by this user
Mark all as spam, and block user from posting to "Tickets"
Originally posted by: birkskyum
Ah, then try this
meteor npm install fibers --saveView and moderate all "tickets Discussion" comments posted by this user
Mark all as spam, and block user from posting to "Tickets"
Originally posted by: birkskyum
Actually, could you try removing chimp? It's causing a lot of trouble for an unused package at this point.
View and moderate all "tickets Discussion" comments posted by this user
Mark all as spam, and block user from posting to "Tickets"
Originally posted by: rsercano
Indeed, will do that and try again
View and moderate all "tickets Discussion" comments posted by this user
Mark all as spam, and block user from posting to "Tickets"
Originally posted by: rsercano
I don't know what's wrong but cant make meteor command work on Monogclient's directory now. Will try that tomorrow morning again it can be about windows, but I'm not sure. Getting error "The system cannot find the path specified" only on Mongoclient directory somehow when I run meteor command
View and moderate all "tickets Discussion" comments posted by this user
Mark all as spam, and block user from posting to "Tickets"
Originally posted by: birkskyum
Is it only when you are in the mongoclient directory, or is it also when you are in other meteor app directories? I tried something similar some time ago, and I ran the meteor installer to fix it.
View and moderate all "tickets Discussion" comments posted by this user
Mark all as spam, and block user from posting to "Tickets"
Originally posted by: rsercano
Only in mongoclient's directory, other than that I'm able to use meteor as usual that happened after
meteor npm installI guess. Will do that thanks a lot, still I need to uninstall meteor from windows before starting installerView and moderate all "tickets Discussion" comments posted by this user
Mark all as spam, and block user from posting to "Tickets"
Originally posted by: rsercano
Hello Birk,
I've just made it work by reinstalling meteor. This seems great, but still there are some issues. First of all could you please remove chimp from your fork as well.
Secondly, some buttons seem to be get smaller somehow. (User management refresh, dump/restore, easy edit etc...) Could you please have a look at it ?
Finally, again autocomplete is not working, expect these PR seems really cool. And I'll accept this PR asap.
View and moderate all "tickets Discussion" comments posted by this user
Mark all as spam, and block user from posting to "Tickets"
Originally posted by: birkskyum
Could you give me an example of the autocomplete issue? I'm not sure where it is.
View and moderate all "tickets Discussion" comments posted by this user
Mark all as spam, and block user from posting to "Tickets"
Originally posted by: rsercano
As soon as you enable it from settings and connect to a database, you should be able to use autocomplete on find/findOne query or more generally where ever you see Selector field. Oh with the keys CTRL+SPACE
View and moderate all "tickets Discussion" comments posted by this user
Mark all as spam, and block user from posting to "Tickets"
Originally posted by: birkskyum
The tests will break from now on because chimp is removed, but the three issues you found should be fixed now.
View and moderate all "tickets Discussion" comments posted by this user
Mark all as spam, and block user from posting to "Tickets"
Originally posted by: rsercano
Checking them now, thank you !
View and moderate all "tickets Discussion" comments posted by this user
Mark all as spam, and block user from posting to "Tickets"
Originally posted by: rsercano
I still cant use autocomplete, am I missing something ?
View and moderate all "tickets Discussion" comments posted by this user
Mark all as spam, and block user from posting to "Tickets"
Originally posted by: birkskyum
Weird. It's working fine here now. Does the console tell you anything?
View and moderate all "tickets Discussion" comments posted by this user
Mark all as spam, and block user from posting to "Tickets"
Originally posted by: rsercano
Unfortunately no, will check again
View and moderate all "tickets Discussion" comments posted by this user
Mark all as spam, and block user from posting to "Tickets"
Originally posted by: birkskyum
Does the code folding work if you write the following?
View and moderate all "tickets Discussion" comments posted by this user
Mark all as spam, and block user from posting to "Tickets"
Originally posted by: rsercano
No, not really, tried to clear cache, but no chance
View and moderate all "tickets Discussion" comments posted by this user
Mark all as spam, and block user from posting to "Tickets"
Originally posted by: birkskyum
Are the 'if' and 'return' colored blue if you insert the following in the findOne query field?
View and moderate all "tickets Discussion" comments posted by this user
Mark all as spam, and block user from posting to "Tickets"
Originally posted by: birkskyum
If they are not blue, then please double-check that you have pulled the last of my commits called:
Load addons (eg. autocomplete) and theme for codemirror
View and moderate all "tickets Discussion" comments posted by this user
Mark all as spam, and block user from posting to "Tickets"
Originally posted by: rsercano
Oh I'm truly sorry, you're a star it's working like a charm. Dumb me forgotten last pull. Accepting this PR now and will find a workaround for travis soon.
View and moderate all "tickets Discussion" comments posted by this user
Mark all as spam, and block user from posting to "Tickets"
Ticket changed by: rsercano