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

add check order method #13

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

AlexTransit
Copy link
Contributor

@AlexTransit AlexTransit commented Nov 16, 2024

добавлен метод проверки заказа
https://www.tbank.ru/kassa/dev/payments/#tag/Standartnyj-platezh/operation/CheckOrder

думал что покажет все операции по заказу, а по факту показывает только последнюю.

был прецедент.
сделал сгенерил QR.
клиент оплатил его.
в процессе выполнения у робота произошла ошибка.
сервер сделал отмену платежа. все прошло без ошибок.
а потом банк отменил возврат и уведомление не прислал. :)

со слов поддержки, типа не было средств на счету. по идее это можно эмулировать.
или в конце дня проверять соответствие статусов на отмененные заказы.

checkorder.go Outdated Show resolved Hide resolved
checkorder.go Outdated Show resolved Hide resolved
checkorder.go Outdated Show resolved Hide resolved
checkorder.go Outdated Show resolved Hide resolved
checkorder.go Show resolved Hide resolved
@AlexTransit
Copy link
Contributor Author

приношу извинения, что создал чтолько проблем. :)
просто я не програмирую. приходиться исправлять/добавлять наследие. :)
исправил все что Вы указали.

checkorder.go Outdated
Status string `json:"Status"` // Статус платежа
RRN string `json:"RRN,omitempty"` // Внутренний номер операции в платежной системе — кроме операций по СБП.
Success bool `json:"Success"` // Успешность операции
ErrorCode uint64 `json:"ErrorCode"` // Код ошибки, «0» - если успешно
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Нужно как-то проверить, какой тут тип поля. Иначе потом придется переделывать в случае ошибки.
Я вижу что например notification использует string, но в доке написано number.

Нужен тест (как описал в предыдущем комменте)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok. я проверю что они возвращают в ErrorCode.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

извините за задержку.
у меня не получилось проверить. я не знаю как сделать заказ с ошибкой в платеже.
возможно что никак.
думаю что надо оставить код ошибки в списке PaymentsCheckOrder как в описании.
ErrorCode | number
и в примере тоже "ErrorCode": 0

а вот ошибка код ошибки в корневом ответе - стока. и проверить легко. поставил несуществующий заказ и получил код ошибки с типом string.

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

Successfully merging this pull request may close these issues.

2 participants