А интерфейс отправки единый. Фишка в том чтобы легко менять сервисы без перепаисывания кода.

Да, но тогда суть идеи теряется.

Ты не сможешь просто так перепрыгнуть с одного на другой сервис потому что у тебя в базе _на проде_ уже формат номеров неверный

пусть подключенный сервис сам формат меняет как ему надо

пусть подключенный сервис сам формат меняет как ему надо

Значит они должны приходить в одном формате все.

У меня просто сейчас concern подмешивается в модели с телефонами для отдачи номера, отформатированного под формат шлюза

я бы сделал возможность выставить дефолтную зону в initializer'e

Я думаю это должно самоп ирложение делать при регистрации пользователя.

Поэтому проблем смены шлюза не будет )

Гемом проще все это, инкапсулированная логика, простой интерфейс и все оттестировано.

Я ща еще хелпер для rspec пишу чтобы легко отправку смс в геме тестировать

Да, полезная вещь

типа, expect { MyService.run(*blah) }.to send_sms.to("+7999").with_text("blah") .and_return(status: :success, meta: :stuff)

круто же ну

А если сделать в конфиге поддержку формата СМС?

Типа хоть в виде регуляок

А если сделать в конфиге поддержку формата СМС?

А как это может выглядеть?

В initializer

вот смотри, ты пилишь реализацию. Чтобы проводить какие то манипуляции с номером, ты должен ЗНАТЬ какой формат номера будет

Значит он должен быть один для всех.

да и так один формат будет, если разработчик не даун

Вида ActiveSms.format = /\d{10}/

да и так один формат будет, если разработчик не даун

Проще застраховать разработчика от возможных проблем.

Вида ActiveSms.format = /\d{10}/

Это теряет всякий смысл т.к реализации не могут нейросеть с собой таскать чтобы ванговать. Им надо знать какой формат вернется.

Лан, буду ArgumentError кидать, надеюсь нормальное решение.

А, точно, ты же еще конвертацию хочешь делать

Верно, у некоторых сервисов формат может быть свой.

У кого то в базе лежат локального формата номера да?

Хм, а почему не хочешь сделать кастомный класс ошибки?

У меня без кода страны хранятся

Ибо пока не надо

Хм, а почему не хочешь сделать кастомный класс ошибки?

Ты должен в development это отловить. В production у тебя уже в модельке юзера должна быть валидация на телефон.

я бы сделал возможность выставить дефолтную зону в initializer'e

В общем годная идея, спасибо. Надо подумать.

Хм, тогда оставь возможность оставить выброса ошибки для прод

Ну у тебя будет ArgumentError кидаться.

Не катит?

А, тогда ок

Провода, наверное было бы понятнее ActiveSms::PhoneFormatError

Или что-то подобное

Наверное на всякий случай лучше действительно кастомную ошибку сделать.

Ладно всем спасибо, пару годных идей для себя усвоил.

Угу, а то вдруг еще какой гем тоже будет аргумент еррор кидать - фиг отловишь нормально )

Тоже верно.

Если будет время - сделаю гем для смс услуг

А что за смс услуги? кинь ссылку?

sms-uslugi.ru

там сейчас есть старый гем, но его поддерживать лень )

Обычно есть апишка на основе которой можно запилить свой гем быстро и просто.

Забавно что на главной я не увидел ни одной ссылки для программистов.

Да, там только после регистрации у них есть документация )

жесть конечно.

https://github.com/drakmail/sms-uslugi-api/blob/master/lib/sms-uslugi-api.rb

Там примерно такое API, реализуется очень просто

Любое нормальное апи быстро пишется.

https://github.com/Fedcomp/active_sms-backend-smsru/blob/master/lib/active_sms/backend/smsru.rb Вот например моя совместимая с active_sms реализация под sms.ru

Это правда не релиз еще.

Правильно делает - это поле используется для STI и свои поля так называть нельзя. Лучше переименуй в kind

Мне нужен тип sti

В жсоне

Мне нужен тип sti

А нет в options что отдаст все атрибуты?

А нет в options что отдаст все атрибуты?

Не знаю

Не знаю

Ну так проверил бы перед тем, как хакать

Ну так проверил бы перед тем, как хакать

Похуй на активрекорд, будет время заменю

Похуй на активрекорд, будет время заменю

Тоже решение, хоть я и не рвусь здесь его пропагандировать. Но уже два-три вопроса (проблемы) с active record (этот, с отношениями у Conversation) имели для меня очевидное решение со стороны Sequel

Sequel хорош во всем

Sequel хорош во всем

не уверен, нужно грамотно с ним работать и разделять логику, но, следуя паттену ActiveRecord, он реализует его намного лучше, чем одноимённый gem

Я уверен

ERROR: S client not available

И не надо следовать паттерну активрекорда

Хранить хоть какую то логику в модели это абсурд

Туда же идет валидация

Все что связано с датасетом - ок в модели

квери обжекты, сервис обжекты, декораторы и форм обжекты отлично интегрируются с активрекордом)

квери обжекты, сервис обжекты, декораторы и форм обжекты отлично интегрируются с активрекордом)

Что значит интегрируются?

* совмещаются, если точнее

модельки становятся только про данные, вся остальная логика разнесена в свои классы

бонусом получаешь реюзабельность и тестируемость +100500 )

модельки становятся только про данные, вся остальная логика разнесена в свои классы

О чем я и написал

Но повторяю что без валидации

Ну да, если использовать active record без фанатизма - то всё ок

Для валидаций форм обджекты только в путь

Активрекорд заставляет меня делать простые вещи сложно

Сейчас использую reform в большинстве проектов - в принципе очень ничего так, хотя иногда и не совсем интуитивно работает

Сейчас использую reform в большинстве проектов - в принципе очень ничего так, хотя иногда и не совсем интуитивно работает

Dry-validation

всё хочу его потыкать, но пока руки не доходят =(

А потом в базе оказываются неконсистентные данные да?

*невалидные

смотря что подразумевается под валидными данными – как правило очень часто оказывается, что в разных кейсах использования моделей разные валидации нужны – тогда в случае толстых моделей у тебя начинают появляться всякие кондишенелы и прочий ад, что в итоге приводит к размытию логики валидаций

ну и плюс валидации, следящие за консистентностью именно можно запихнуть в модель, как правило их не так много

у вас рельсы в головах

РГМ?)

"И горжусь" ?

Подскажите пожалуйста, как убрать возможность входа на страницу регистрации gem devise?

для незарегистрированных юзеров

Подскажите пожалуйста, как убрать возможность входа на страницу регистрации gem devise?

Еще раз но по русски

Ты хочешь выключить регистрацию для незарегистрированных юзеров?

Подскажите пожалуйста, как убрать возможность входа на страницу регистрации gem devise?

убрать registerable из конфигов, если по-простому

Ну ему же именно для незарегистрированных юзеров

Посмотри в Вики у них. Вроде там было