Как реализовать sms-подтверждение регистрации?

Question

[askhat]

Сайт построен на Sintra и в экшенах есть такой код:

post '/registration/sms/:number' do
  code = Random.rand(1111..9999).to_s
  phone = params[:phone].to_s
  HTTParty.get('http://sms.ru/sms/send?api_id=' + api_id +'&to=' + phone + '&text=' + code)
  # check code
end

И такая вьюха:

%form{:method => "post"}
  %div.form-group
    %label Телефон
    %input#phone.form-control{:name => "phone", :placeholder => "7 950 123 45 67"}
    %p.help-block Введите доступный Вам номер, на него будет отправлен код активации
  %div.checkbox
  %button#sendsms.btn.btn-default{"data-toggle" => "modal", "data-target" => "#myModal"} Отправить

С таким JavaScript-ом:

$("#sendsms").click(function(){
  var phone = $("#phone").value();
  $.ajax({
    url: "/registration/sms",
    data: {"phone": phone},
    type: "post"
  })
})

По разумению моему, JS должен отправлять post запрос на /registration/sms с параметром phone, который обрабатываясь экшеном, создает новый запрос на sms.ru, который собственно отправляет sms. Однако этого не происходит и я ума не приложу как заставить его работать.
Заранее благодарю за любую помощь.

Answer 1

[askhat]

В экшене не нужен параметр :number, так что он должен выглядеть так:

post '/registration/sms' do
  # code..
end

Также необходимо исправить присваивание переменной phone в JS, вот так:
var phone = $("#phone").val();

Answer 2

[Eugene Obrezkov]

Первое что бросается в глаза:
Принимаете по такому роуту - post '/registration/sms/:number' do
А отправляете по другому url: "/coop/sms".

Без отладки не поможем. Причин может быть множество: повисла апишка смс-ок, на бекенде не приходят параметры, на фронтенде не берет параметры.

Comments

[askhat]

Пофиксил registration, ошибся пока переписывал.

[Eugene Obrezkov]

@happy_buddha а попробуйте сделать пост запрос по урлу registration/sms/12345678910, то есть не передавая дату.
$("#sendsms").click(function(){
var phone = $("#phone").value();
$.ajax({
url: "/registration/sms/" + phone,
type: "post"
})
})

[askhat]

Вот это "code = Random.rand(1111..9999).to_s" с вот этим "HTTParty.get('sms.ru/sms/send?api_id=' + api_id +'&to=' + phone + '&text=' + code)" работают в irb, следовательно апишка не повисла.

[askhat]

@ghaiklor не работает также

[Eugene Obrezkov]

@happy_buddha тогда зачем там в роуте placeholder, непонятно :) По поводу без JS. Во вьюхе добавьте форме action атрибут с указанием вашего урла, куда идет пост.

[Eugene Obrezkov]

@happy_buddha тогда тот жс можете смело убирать. Добавляете урл к форме, добавляете input type=submit. И начинайте смотреть в код того action'а на бекенде. Принимаете ли вы вообще эти параметры и т.п.

[askhat]

@ghaiklor хотелось бы до отправки остальных параметров формы, вызвать модальное окно, в котором бы проходила валидация смс. Так что вопрос "без JS" снят.

[Eugene Obrezkov]

@happy_buddha в любом из вариантов ответ с проверкой "что происходит у вас на бекенде" не снят. Смотрите туда и отлаживайте, почему не идет запрос или данные. Тут ничего не поможешь через QaA.