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

Meteor package update #769

Closed
92hackers opened this issue Dec 9, 2015 · 47 comments
Closed

Meteor package update #769

92hackers opened this issue Dec 9, 2015 · 47 comments
Milestone

Comments

@92hackers
Copy link

I'am using this project in my meteor project, and i found the version is still 0.3.0, which is too old,
there is big difference between demo and my developed project.
maybe you have forgotten updating it or something others.
wish that, and i am willing to help : ) !

@rosskevin
Copy link
Contributor

Actually I am unfamiliar with meteor all together. @FezVrasta is there something I need to do to make this available via meteor and npm? I'm primarily just making sure good tags exist for bower and updating the release notes.

@92hackers
Copy link
Author

i have already make the meteor package, code from tag v0.4.5, if i should publish it to the atmospherejs.com ?

@FezVrasta
Copy link
Contributor

I think there was a grunt task to update the atmospherejs package... I've not experience with it unfortunately.

@rosskevin
Copy link
Contributor

@FezVrasta if you will add rosskevin to the atmospherejs and npmjs sites as a project admin, I'll try to push and see what happens.

@rosskevin rosskevin added this to the 0.5.5 milestone Dec 9, 2015
@FezVrasta
Copy link
Contributor

I lost the password of the account associated with the project :|

edit: actually I can login to my account but I can't find a way to add a collaborator...

@rosskevin
Copy link
Contributor

It may give me a good message once it is publishable. @92hackers I need you to try and fix the errors and submit a PR. On master I have updated some of the files, but I'm lost otherwise and I admit I don't have a lot of motivation on this front.

If you can make grunt meteor-test work and submit that as a PR, I should be able to attempt to publish.

@rosskevin rosskevin changed the title Maybe you have forgotten updating this project in atmospherejs.com( meteor package) Meteor package update Dec 9, 2015
@rosskevin rosskevin removed this from the 0.5.5 milestone Dec 9, 2015
@92hackers
Copy link
Author

got it, i will do it as soon as possible

@92hackers
Copy link
Author

@rosskevin , can you explain me about meteor/test.js ?
code : var plugins = ["affix", 'alert', "",,,,,,,,];
but , where do you include those plugins ?
and why test those plugins ?

@rosskevin
Copy link
Contributor

I have no idea why they were included, but they are BS plugins, so perhaps it was testing that or it was just copies and never fixed. It doesn't seem like a good test of MDB.

@rosskevin
Copy link
Contributor

@FezVrasta here is a note on adding contributors/publishing - it recommends creating an organization with people:
http://stackoverflow.com/a/29269583/2363935

I think we either need to get this updated or pull it, people are using a package that is out of date and it makes it look like the project is not working (which it may indeed not be for meteor users).

@FezVrasta
Copy link
Contributor

I would drop it at all... If someone wants to maintain it, he will create a fork I guess.

@rosskevin
Copy link
Contributor

I don't have access to drop it on atmosphere. I would suggest you drop it (and I'll drop the related grunt code because it doesn't work). If someone steps up and submits a PR, we can go from there.

@FezVrasta
Copy link
Contributor

I wonder how should I be able to drop it from atmosphere... The account linked to it is fezvrasta, but I have the credentials to fezvr4sta... when I try to login to fezvrasta it says that such account has not a password set :/

@dandv setup it for me ages ago... If I remember correctly

@rosskevin
Copy link
Contributor

If @dandv cannot help pull it, perhaps submit a support request?

@FezVrasta
Copy link
Contributor

Let's wait a day to see if he can help us, if not, I'll then submit a ticket.

@tsega
Copy link
Contributor

tsega commented Feb 3, 2016

@FezVrasta I must say you have one of the most popular packages on Meteor some 7K+ downloads. I myself use if for almost all of my new projects but I didn't realize it was totally out of sync with the original.

I want to help getting this up to date:

I lost the password of the account associated with the project :|
edit: actually I can login to my account but I can't find a way to add a collaborator...

Here is what you need to do,

  1. Go to meteor.com and sign in with your account, i.e fezvrasta.
  2. Then go to you profile settings to create an organization, https://www.meteor.com/account-settings/organizations ; this way you could add a trusted person like @rosskevin to your organization and he can follow with updating etc.

Note: @rosskevin would also need a meteor.com account.

I would also love to help with this!

@rosskevin
Copy link
Contributor

I have a meteor account - it is rosskevin.

@tsega we would love to have your help. What we need right now is a PR that makes the grunt meteor tasks work. It doesn't appear that the grunt meteor-test has any bearing on this library, and may have been copied.

Permissions aside, I just need a PR from a meteor user that ensures I can call grunt meteor when I a have a new version to release.

@FezVrasta
Copy link
Contributor

@tsega, sorry but I get this when I try to login to fezvrasta:

image

If I try to login with my real account fezvr4sta, I can login but I have no access to the repo settings.

@tsega
Copy link
Contributor

tsega commented Feb 3, 2016

OK it seems like the user fezvrasta is not a simple user but an organziation.

screen shot 2016-02-03 at 8 21 24 pm

Your real user is fezvr4sta. Use that to login.

@FezVrasta
Copy link
Contributor

Ah! yes!

I've added rosskevin to the collaborators

@tsega
Copy link
Contributor

tsega commented Feb 3, 2016

👍 @rosskevin I'll be working on that PR, but I must admin I haven't used grunt, gulp or bower. I'm a total Meteor fanboy 😊

@FezVrasta
Copy link
Contributor

I think we just need a task to pubblish the new builds on atmosphere...

@tsega
Copy link
Contributor

tsega commented Feb 3, 2016

That's so strange my copy doesn't have that folder, I have your latest commit as well. I'll try to clone again.

@rosskevin
Copy link
Contributor

A clean could have run, i.e. if you ran grunt. Just run grunt dist

@tsega
Copy link
Contributor

tsega commented Feb 3, 2016

Yes that's correct I did grunt clean earlier. So now I'm getting a very similar error to this one, https://travis-ci.org/FezVrasta/bootstrap-material-design/builds/89655229

@rosskevin
Copy link
Contributor

Yes, these are the meteor tests, which appear to be bogus.

@tsega
Copy link
Contributor

tsega commented Feb 3, 2016

Could this be something related, themeteorchef/jquery-validation#6

@rosskevin
Copy link
Contributor

I have no idea. I'm not sure if I made this clear before, but I think these tests should be deleted because they appear to have been copied from bootstrap.

If you aren't going to make them relevant to MDB, just delete them is my thought.

@tsega
Copy link
Contributor

tsega commented Feb 3, 2016

Got the bugger! As I suspected it was a jquery issue. I had to add

$ =jQuery.noConflict();

At the top of the meteor/test.js file on the second line right under user strict;

screen shot 2016-02-03 at 11 23 48 pm

What's next?

@rosskevin
Copy link
Contributor

Try grunt meteor. If you make it up to a publishing error due to credentials, I think that's all you can do, you just need to submit your PR

tsega added a commit to tsega/bootstrap-material-design that referenced this issue Feb 3, 2016
@tsega
Copy link
Contributor

tsega commented Feb 3, 2016

As you expected, everything passed but I was unable to publish due to credentials. Here you go:
screen shot 2016-02-03 at 11 30 03 pm

@tsega
Copy link
Contributor

tsega commented Feb 3, 2016

@rosskevin I've submitted my PR.

rosskevin added a commit that referenced this issue Feb 3, 2016
Fixed Meteor tests failing because of jQuery conflict. Related #769
@rosskevin rosskevin added this to the 0.5.8 milestone Feb 3, 2016
@rosskevin
Copy link
Contributor

Published, thank you @tsega

@tsega
Copy link
Contributor

tsega commented Feb 3, 2016

Thank you @rosskevin for the quick publish! I'm the first one to download it 😄 Please let me know if you need my help with any of the remaining issues.

@rosskevin
Copy link
Contributor

The only problem remaining is that there are 4 packages now, 3 of which are no longer relevant:
https://atmospherejs.com/fezvrasta

I cannot seem to delete these others.

@tsega
Copy link
Contributor

tsega commented Feb 3, 2016

Ah! That's the only thing I don't like about atmosphere, once published you can't remove a package. They say it has to do with keeping backward compatibility or something. I'm afraid there is nothing we can do about that.

@dandv
Copy link
Contributor

dandv commented Feb 3, 2016

Hey guys. Looks like you've sorted out the permissions.

I haven't followed the latest developments, so I'm not sure why the -noglyph package is no longer relevant, but if you want to hide it, run

meteor admin set-unmigrated <package:name>

@rosskevin
Copy link
Contributor

@dandv - unfortunately that appears to have failed as well:

$ meteor admin set-unmigrated fezvrasta:bootstrap-material-design-fullpalette
done.                                         [email protected] as unsuccessfully migrated ...                                               
done.                                         [email protected] as unsuccessfully migrated ...                                               
$ meteor admin set-unmigrated fezvrasta:bootstrap-material-design-noglyph
done.                                         [email protected] as unsuccessfully migrated ...                                               
done.                                         [email protected] as unsuccessfully migrated ...                                               
done.                                         [email protected] as unsuccessfully migrated ...                                               
done.                                         [email protected] as unsuccessfully migrated ...                                               
$ meteor admin set-unmigrated fezvrasta:bootstrap-material-design-noicons
done.                                         [email protected] as unsuccessfully migrated ...                                               
done.                                         [email protected] as unsuccessfully migrated ...                                               
done.                                         [email protected] as unsuccessfully migrated ...                                               
done.                                         [email protected] as unsuccessfully migrated ...   

@rosskevin rosskevin reopened this Feb 4, 2016
@tsega
Copy link
Contributor

tsega commented Feb 5, 2016

@rosskevin actually that has worked, if you look at your teams profile on atmosphere the only package now is fezvrasta:bootstrap-material-design. The message as unsuccessfully migrated is what it's supposed to say.

@dandv thanks for the command been looking for something like for a while now. I think it's not being shared with the masses for a reason, right?

@rosskevin
Copy link
Contributor

👍

@tsega
Copy link
Contributor

tsega commented Feb 11, 2016

Sorry if it's wrong to ask this here but the original problem that lead me here is still bugging me.

On my Android device running 4.1.2, the checkboxes appear but neither the ripple effect or the actual check mark appear when checked. How can I get this to work?

Even more, check boxes that don't have labels cannot be selected. I have the same issues on the iOS emulator but not the android one. Pretty strange, if you ask me!

screenshot_2016-02-11-12-16-09

@rosskevin
Copy link
Contributor

@tsega please raise a different issue. We cannot log each and every issue for a meteor issue here.

My initial advice is to log a new issue, and ask for advice. You should know that you are likely to be the one that fixes it, but by logging a separate issue and asking, the community may be able to help you.

On a personal note from someone with deep experience in mobile, please first look at the mobile market share and make sure it is a must-have. While it would be nice to fix this, the mobile market upgrades quickly.

@mdbootstrap mdbootstrap locked and limited conversation to collaborators Feb 11, 2016
@mdbootstrap mdbootstrap locked and limited conversation to collaborators Feb 11, 2016
@rosskevin
Copy link
Contributor

Meteor guys - see #930, we have officially dropped support and need someone to step up and maintain a package. We will be happy to link to you.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants