DispatchMe/meteor-accounts-sms

carrier always null on sms client.lookup()?

Opened this issue · 0 comments

It appears that the sms client.lookup would never throw an error as the "Carrier" option is not passed, thus lookup.carrier will always be null. See: https://github.com/DispatchMe/meteor-accounts-sms/blob/master/sms_server.js#L70-L71.

I think it should be var lookup = Accounts.sms.client.lookup(phone, {Type: 'carrier'});. Would create a PR but haven't used this package yet so don't know if it's an active issue yet, but thought I should mention it.