Изменения в README.rst #1
Conversation
1. Additional strict check description added. 2. Own form necessity added. 3. Own view necessity added.
| @@ -63,34 +63,138 @@ ROBOKASSA (http://robokassa.ru/Doc/Ru/Interface.aspx). Приложение ре | |||
| Форма для приема платежей | |||
| ------------------------- | |||
|
|
|||
| Для того, чтобы упростить конструирование html-форм для отправки пользователей в | |||
| Для того, чтобы упростить отправку и валидацию данных в | |||
There was a problem hiding this comment.
RobokassaForm не выполняет валидацию данных.
|
Привет! Оставил комментарии к коммитам. Там 2 изменения, насколько я понял:
+1 ко второму имненению. Про первое я бы еще подумал - он какой-то неминимальный получился (ну и по стилю замечания всякие - учить пользователя регекспам и HTML5 не стоит в справке к django-robokassa, там и так много информации). Раньше был пример с "оплатой товара", стал пример с "пополнением счета". Можешь немного больше мотивацию пояснить? Такие навороты обычно сигнал к тому, что с приложением что-то не так, и его улучшить можно; если это тот случай, то я бы предпочел приложение как-то улучшить, а не документацию усложнять. |
|
Необходимость второго изменения объяснил мне наш более опытный в вопросах биллинга джедай после того, как я сделал все без ResultUrl. Сайты для серьезного бизнеса =) действительно не обойдутся без двойного подтверждения перевода средств. На первом изменении не настаиваю, я чуть поменял и скопипастил то, как делал это у себя. Товары у нас не продаются, только услуги, поэтому я работал не с "оплатой товара", а с "пополнением счета". Это, конечно, надо поменять. Вобщем, "правьте смело", как говорят в вики, и оставляйте те изменения, которые считаете нужными. |
|
Спасибо! Я пока в README добавил то, по чему вопросов не возникло. Пулл-реквест пока оставляю открытым. |
В readme:
Если (и наверняка) в изменениях есть что-то негодное, то могу поправить или Вы поправьте. Я старался писать с учетом того, как это воспринимает человек, который в первый раз использует django-robokassa и в первый раз пишет биллинг, то есть не знаком с механизмами подтверждения платежей, не знает, по каким причинам они нужны для безопасности.