Merge branch '658-check-numbers-sent-to-rising-tide' into 'master'

Fix bugs found when testing with app. #658

Closes #658

See merge request jankstudio/resq!771
This commit is contained in:
Kendrick Chan 2022-04-12 11:03:24 +00:00
commit 19a757f48d

View file

@ -249,7 +249,7 @@ class APIController extends Controller implements LoggedController
$rt->sendSMS($phone_number, $translator->trans('message.battery_brand_allcaps'), $message); $rt->sendSMS($phone_number, $translator->trans('message.battery_brand_allcaps'), $message);
} }
public function confirmNumber(RisingTideGateway $rt, Request $req) public function confirmNumber(RisingTideGateway $rt, Request $req, TranslatorInterface $translator)
{ {
// check parameters // check parameters
$required_params = [ $required_params = [
@ -306,7 +306,7 @@ class APIController extends Controller implements LoggedController
if ($otp_mode != 'test') if ($otp_mode != 'test')
{ {
// send sms to number // send sms to number
$this->sendConfirmationCode($rt, $phone_number, $code); $this->sendConfirmationCode($rt, $phone_number, $code, $translator);
} }
// response // response