carrier always null on sms client.lookup()?
Opened this issue · 0 comments
spacemunkay commented
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.