em0ney / meteor-timezone Goto Github PK

The idea of this package is good, but unfortunately it neither supports audit-argument-checks nor SimpleSchema. So if you're using one of them, it throws errors: SimpleSchema complains that the user profile schema does not contain the timezone field, and audit-argument-checks does complain that the method call does not check the input parameter.

SimpleSchema support can be added easily on the app side, and I think this should also be the duty of the app developer. Yet, it would be nice if the package docs stated which data format is used for the timezone (I think it's just a Number).

The audit-argument-checks problem, however, cannot be solved on the app side, since it has to be done in the method code. So I would recommend to add a check() in the method. For now, I will have to do a fork of this package to make it work in my app.

BTW: I was also missing a license information and a link from Atmosphere to Github.