Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

robokassa + tweaks = russian_adaptation? #3

Open
konzeptual opened this issue Jan 5, 2010 · 4 comments
Open

robokassa + tweaks = russian_adaptation? #3

konzeptual opened this issue Jan 5, 2010 · 4 comments

Comments

@konzeptual
Copy link

Доброго времени.
Спасибо за то, что делитесь своей работой!

Предложение у меня такое - может быть стоит разбить функционал расширения?
К примеру, было бы неплохо иметь функционал Робокассы в отдельном расширении. Это удобно, если хочешь пользоваться Робокассой, но при этом иметь альтернативный процесс оформления заказа.

Если у вас нет времени, то я бы это сам сделал, мне все равно придется выделить робокассу в отдельное расширение. Работы должно быть не так много...

@romul
Copy link
Owner

romul commented Jan 5, 2010

Это есть в планах. Только там не столько выделять, сколько переписывать надо. Теперь в Spree есть BillingIntegration как раз для подобных целей.

@konzeptual
Copy link
Author

Посмотрел на новый механизм BillingIntegrtaion.
С ним действительно выделение робокассы в отдельное расширение совсем не обязательно.

@romul
Copy link
Owner

romul commented Jan 11, 2010

Да, я потихоньку начал обновление.
План пока такой:

  1. Перевести РобоКассу на гейтвэй ActiveMerchant, т.к. он уже есть готовый, спасибо parallel588. (сделано)
  2. Изменить контроллер PaymentsController, надо дать ему более специфичное имя, чтобы было понятно, что это контроллер для RoboKassa и генерить ошибку 404 или 500, если текущий гейтвэй не является РобоКассой.
  3. Посмотреть есть ли смысл переделывать на BillingIntegration или достаточно оставить в качестве Gateway. Основная разница между ними, что гейтвэи работают через ActiveMerchant, а BillingIntegration может быть абсолютно независимым.

@konzeptual
Copy link
Author

Руки наконец дошли до робокассы.
Сделал следующим образом:

  1. Оставил робокассу гейтом
  2. Процесс заказа такой:
    Адрес => вид доставки => вид оплаты => заказ формируется, имеет статус "новый", теперь покупатель может нажать кнопку "оплатить" и иметь дело с робокассой (если он перед этим выбрал робокассу в качестве вида оплаты).

http://github.com/krick/spree-russian-adaptation/tree/robokassa

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants