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

Package depending on deprecated aldeed:simple-schema #108

Open
danielparas opened this issue Aug 10, 2017 · 4 comments
Open

Package depending on deprecated aldeed:simple-schema #108

danielparas opened this issue Aug 10, 2017 · 4 comments

Comments

@danielparas
Copy link

Hey @jonjamz! as usual thanks for this package - it's been very useful!

I just updated aldeed:collection2 and it was quite a major update. Part of which involves the removal and deprecation of the aldeed:simple-schema package. Please see below, note from the package author:

Verify that aldeed:simple-schema is not listed in your .meteor/.versions file. If it is, you likely depend on some other package that has an explicit dependency on the Meteor aldeed:simple-schema package. This could cause problems or unexpected behavior. Work with that package author to get an updated version that does not explicitly depend on the aldeed:simple-schema package.

From:
https://github.com/aldeed/meteor-collection2

Is it possible to remove the aldeed:simple-schema dependency?

Thanks
Daniel

@jonjamz
Copy link
Owner

jonjamz commented Aug 11, 2017 via email

@danielparas
Copy link
Author

Thanks for the quick reply @jonjamz ! If I can help in any way let me know

@jonjamz
Copy link
Owner

jonjamz commented Aug 14, 2017 via email

@danielparas
Copy link
Author

danielparas commented Aug 21, 2017

Hey @jonjamz - I never really managed to test it out tbh, as the application I'm using your package on also has another package (tap:i18n) that uses simpleSchema and it's throwing a simpleSchema not found error stopping the app from loading. This is what lead me to research which packages are using simpleSchema. tap:i18n and templates:forms are the only two I'm using and in the case of tap:i18n the issue has already been reported. So i thought might as well give a heads up here also.

Re: testing - Disabling the tap:i18n plugin, to test template:forms with the new simpleschema would cause the application to break, unfortunately. In the mean time I tried setting up a separate test application but I'm having trouble getting templates:forms to work, however I'm not sure if it has anything to do with simpleschema as the error I get is "Cannot set property 'data' of undefined" (the test app is just a copy of the example scripts you have in the package documentation i.e. basicFormBlock + basicinput + associated helpers.)

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

No branches or pull requests

2 participants