-
Notifications
You must be signed in to change notification settings - Fork 187
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
Meteor 1.2 Compatibility #383
Comments
Having the same issue as mentioned above, not sure why the package is listed as incompatible with meteor 0.9.0 or after - been using it with 1.1 for a while. |
that's weird actually! |
I don't know why, but in upgrading to Meteor 1.2, accounts-entry was downgraded from 1.0.3 (which I've been using with no issues) to 0.7.3. Meteor then reports accounts-entry 0.7.3 is incompatible with Meteor 0.9.0 or later. When I attempt to manually update accounts-entry to 1.0.3, I see messages re: conflicting LESS constraints. |
Same here |
same here... |
same. Need a fix... |
same issue as the rest |
same |
I faced similar issues, and I what I did was
|
Nope. Uncommenting joshowens:accounts-entry, after upgrading Meteor to 1.2 doesn't actually let the package run, it's marked as incompatible and disabled.
|
I;m sorry I had it locally and updated also the dependancy on meteor version. |
No worries you aren't misleading. Just corrections where warranted ;)
|
I was able to get this package working on 1.2 by cloning the repo into the packages direcory of my project, renaming it, bumping a few version nubers and installing it from my package. Here are my steps:
I updated the project to meteor 1.2 earlier. Also make sure that you remove the normal version of accounts-entry from your project. This is just a temporary solution- we should wait unitil the autor adresses the issue. |
@jakubbilko Could you public your modified package? |
It's here: https://github.com/jakubbilko/accounts-entry |
@jakubbilko Thank you very much! |
@jakubbilko Thanks! Much appreciated. |
@jakubbilko, this did not work for me, still getting the same error message :( |
@greyarch have you removed the old version of accounts-entry from your project? |
@greyarch I had the same issue and it worked after I added the check and ejson packages ( meteor add check ejson ) |
@jakubbilko: thank you very much! Well done. |
I am happy to take a PR and push a release here. |
PR created ;) |
I can see that the PR is now merged - could you please push a release? :) |
In the interim, is it possible to reference @jakubbilko 's package from Atmosphere? |
@JogoShugh why not just git clone it to your project |
I know how to clone a github repository but am not sure sure how to run it to fix this problem. Is someone willing to share the steps needed? Thanks |
@dougwells in my projects root folder i believe that was all i did i can double check if that dosent work when i return home and let you know if you run into problems |
Thanks. Getting closer but still not there. Here is where I am at:
Question: How do I get meteor to install the new/forked jakubbilko version of accounts-entry that I have stored locally in my projects/package.js file? |
Got it!!!!! One small change to @TheSpitefulOctopus excellent counsel. Within my project root folder, I needed to create a folder called packages (instead of projects). After that it worked perfectly. For more information on running packages from your local drive see http://stackoverflow.com/questions/18328000/installing-a-local-package-on-meteor Thanks for everyone's help on this forum. Took ~2 frustrating hours to get it to work but I learned a TON through the process ... |
@queso 👍 could you publish this to atmosphere if it works? |
anyone having luck? I'm stuck also |
Procedure by @TheSpitefulOctopus + comment by dougwells that the folder is named packages (at same level as 'client' and 'server') with single subfolder: the cloned accounts-entry package works OK for now. |
Any chance this is getting fixed without any workarounds? |
@queso - Any chance that merged PR will be published to Atmosphere soon? 😄 |
I'm also getting this error with Meteor 1.2.1:
and:
( I'm new to Meteor and not sure if this (via SO) is the ideal solution, but it seems to have fixed it in the meantime:
|
This works for me:
|
@jakubbilko Thanks a bunch for your fix. =) |
thanks |
For anyone who is doing udemy course on meteor and is having an issue and can't fix it, here is what you need to do (assuming you have not installed the joshowens package yet):
After that meteor will detect the folder and do the work itself. |
It works thanks to @BarinsGhost, @dougwells and @TheSpitefulOctopus |
it works for me too thanks to @zulhfreelancer @BarinsGhost, @dougwells and @TheSpitefulOctopus |
just updated to Meteor 1.2 and hit the same issue reported by @RanaivosonHerimanitra here: #243 (comment)
per the 1.2 Breaking Changes (https://github.com/meteor/meteor/wiki/Breaking-changes-in-Meteor-1.2) I'm guessing LESS constraints need to be updated.
The text was updated successfully, but these errors were encountered: