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

پیشنهاد اضافه کردن متدهای خاص درگاه ها #179

Open
ariaieboy opened this issue Feb 16, 2023 · 1 comment
Open

Comments

@ariaieboy
Copy link
Contributor

ariaieboy commented Feb 16, 2023

سلام خدمت شما
پکیج مولتی پی برای همه درگاه ها متد های عمومی مانند purchase و pay و verify را به کاربر میده که برای بسیاری از پروژه ها همین موارد کافی هست.
اما بعضی از درگاه ها متد های خاصی دارند که مختص همان درگاه می باشد. به عنوان مثال متد پرداخت اشتراکی یا تسهیم پرداخت روی یکسری درگاه های خاص مثل زرینپال پی پینگ و... وجود داره.
یا به طور مثال درگاهی مثل زرین پال به صورت خودکار پرداخت های وریفای نشده را برگشت نمیزنه و باید از طریق متد های unverified و refund این پرداخت ها بررسی بشه و در صورت نیاز بازگشت وجه بخوره. که خوب این موارد در این پکیج پوشش داده نشده است.
یکسری از این متد ها را مانند تسهیم پرداخت یا ریفاند زرین پال را بنده میتونم پیاده سازی کنم و مرج ریکوئست بزنم. اما دو مسئله وجود دارد.
1- آیا این پکیج مرج ریکوئست برای متد های خاص درگاه ها قبول میکند ؟
2- اگر مشکلی در اضافه کردن متد های خاص وجود نداره آیا روش خاصی برای پیاده سازی باید در نظر بگیریم یا خیر ؟
چراکه همه درایور ها از یک اینترفیس یکسان استفاده میکنند و خوب پیاده سازی متد های خارج از اینترفیس میتونه یک مقدار چلنجی باشه چه از نظر داکیومنت کردنش چه از لحاظ نام گذاری متد ها و...

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

@ariaieboy ariaieboy changed the title پیشنهاد اضافه کردن ویژگی های خاص درگاه ها پیشنهاد اضافه کردن متدهای خاص درگاه ها Feb 16, 2023
@khanzadimahdi
Copy link
Member

وقت شما بخیر.
شما میتونید یکسری اینترفیس مجزا تعریف کنید مثلا refundable که مدت refund رو implement کنه و PR بزارید تا مرج کنیم.
دلیل اینکه همه درگاه ها یک اینترفیس یکسان رو پیاده سازی کردن این هست که همه این اینترفیس رو پوشش میدن! اگه متد خاصی مد نظرتون هست که یک درگاه پوشش میده میتونید اونو پیاده سازی کنید و درخواست مرج بزارید (حتی بدون تعریف اینترفیس هم اوکی هست)

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