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 Custom Methods to Digipay Gateway #231

Merged
merged 7 commits into from
Jul 11, 2024

Conversation

alissn
Copy link
Contributor

@alissn alissn commented Mar 10, 2024

Hi,

This pull request adds two custom methods to the Digipay Gateway:

Additionally, only the base URL is defined in the config file, while sub-URLs are added as constants in the class.

This pull request should be merged after #230.

@alissn alissn force-pushed the AddCustomMethodDigipay branch 2 times, most recently from d764055 to 734ac1c Compare March 10, 2024 11:34
@alissn alissn force-pushed the AddCustomMethodDigipay branch 2 times, most recently from 5baea16 to edc6d63 Compare July 2, 2024 14:07
@alissn alissn requested a review from khanzadimahdi July 2, 2024 14:15
@alissn
Copy link
Contributor Author

alissn commented Jul 2, 2024

@khanzadimahdi

بعضی از متد هایی که تو این درگاه اضافه شده و به صورت public هستن، به خاطر اینه که درگاه دیجی پی نیاز داره که بعد از پرداخت موفق صدا زده بشن. چون این درگاه درگاه پرداخت قسطی هستش، بعد از تحویل محصول باید به درگاه گفته بشه که محصول تحویل شد تا اقساط شروع بشه.

یا یه سری متد ها هستن که وقتی سفارشی کنسل میشه، باید به درگاه اعلام بشه که از اقساط رو حذف و اعتبار کاربر رو برگردونن.

ما این کلاس تو یکی از پروژه ها داریم استفاده میکنیم، که روزی ۵ ۶ مورد از طریق این درگاه خریداری میشه، و همچنین چند مورد کنسلی هم چک کردیم هیچ مشکلی نداشته. (البته که به خاطر تاخیر تو مرج شدن این مرج ریکوست، کلاس خودمون رو تو تنطیمات جایگزین کرده بودیم)

@alissn alissn mentioned this pull request Jul 11, 2024
9 tasks
@khanzadimahdi
Copy link
Member

من این کامیت رو مرج میکنم تا مشکلات درگاه فیکس بشه اما در کل اکسپوز کردن متدهایی که جزو اینترفیس نیستن کار درستی نیست و باید یه فکر اساسی راجبش بکنیم.
میتونیم یه میتینگ باهم ست کنیم و راجبش صحبت کنیم اگه موافق باشی.

@khanzadimahdi khanzadimahdi merged commit 1fb4410 into shetabit:master Jul 11, 2024
2 checks passed
@alissn
Copy link
Contributor Author

alissn commented Jul 11, 2024

@khanzadimahdi
بله حتما، خیلی کمک میکنه که بتونیم یه راه اصولی برای درگاه هایی که متدهای خاص دارن پیدا کنیم که لاجیک کلی پکیج هم عوض نشه.

یه راه ارتباطی بفرستین تا یه تایمی برای میت با هم ست کنیم.(پروفایلتون رو نگاه کردم، چیزی پیدا نکردم)

@khanzadimahdi
Copy link
Member

@alissn my telegram id: mahdi.khanzadi
please reach me to schedule a meeting.

This pull request was closed.
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