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 AsanPardakht getway support #121

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

miladkoohi
Copy link

@miladkoohi miladkoohi commented Sep 17, 2024

سلام مجدد @birddevelper

در این برنچ موارد رو رفع کردم امیداورم مرج کانفلیت نداشته باشه و پول انجام بشه با تشکر
داکیومنت جدید پیوست شد
AsanPardakht-IPG-.V1.9.1.pdf

@birddevelper
Copy link
Contributor

@miladkoohi ممنون از شما.
به نظر میاد هنوز مشکلاتی وجود داره که باید رفع بشه، برخی شون رو اینجا مینویسم ولی چون از مرحله ی دریافت توکن جلو تر نرفت، مشکلات مراحل بعدی رو لطفا در نظر بگیرید:

  1. کلاس BankGatewayRejectPayment که در کد مورد استفاده قرار گرفته باید در init ماژول اکزپشن ایمپورت بشه
  2. فراخوانی اندپوینت دریافت توکن با خطای 477 مواجه میشه.

- Resolved issue with missing 'username' attribute in AsanPardakht class by ensuring proper variable assignment.
- Modified the  method to correctly set , , and  attributes.
- Implemented proper error handling in  method for better handling of HTTP errors and exceptions.
- Refactored the  method to retrieve server time from Asan Pardakht API.
- Enhanced logging to provide better debug information during payment process and error scenarios.

Related to issue ali-zahedi#119
- Resolved issue with missing 'username' attribute in AsanPardakht class by ensuring proper variable assignment.
- Modified the  method to correctly set , , and  attributes.
- Implemented proper error handling in  method for better handling of HTTP errors and exceptions.
- Refactored the  method to retrieve server time from Asan Pardakht API.
- Enhanced logging to provide better debug information during payment process and error scenarios.

Related to issue ali-zahedi#119
@miladkoohi
Copy link
Author

با سلام کامیت جدید ایجاد شد روی سرور تستی که درگاه واقعی روش هست تست شد و موارد مشکلی نداشت
امیدوارم تو بررسی ها مشکلی نداشته باشه

@birddevelper
Copy link
Contributor

سلام @miladkoohi . ممنون از تغییرات تون.
موارد تا ارسال کاربر به درگاه بانک برطرف شده. فقط موقع برگشت کاربر از درگاه، اطلاعات پرداخت از دیتابیس موقع واکشی اطلاعات پرداخت از دیتابیس به خطا میخوره که به نظر به خاطر ست نشدن یکی از پارامتر های reference_number یا tracking_code هست.

self._bank = Bank.objects.get(
Q(Q(reference_number=self.get_reference_number()) | Q(tracking_code=self.get_tracking_code())),
Q(bank_type=self.get_bank_type()),
)

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