supabase / auth / 10238183895

69%

Build Type

push

github

Committed by web-flow

Commit Message

fix: allow enabling sms hook without setting up sms provider (#1704)

## What kind of change does this PR introduce?
* Resolves issue where the custom SMS hook cannot be used unless a SMS
provider is configured by moving `GetSmsProvider` into
`sendPhoneConfirmation` and only calls it if the hook is not enabled.
* Allows `channel` to be set for MFA (phone) if hook is enabled 

## What is the current behavior?
* It's not possible to set up a hook without adding config for a SMS
provider

## TODO
- [x] Fix broken tests

Run Details

36 of 54 new or added lines in 8 files covered. (66.67%)

3 existing lines in 2 files now uncovered.

9190 of 15748 relevant lines covered (58.36%)

54.93 hits per line