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 bank factory interface duo to make creation of bank factory cleaner and shorter #116

Merged
merged 5 commits into from
Oct 24, 2024

Conversation

mojtabaakbari221b
Copy link
Contributor

برای ساختن یک instance از روی bank factory، نیاز به نوشتن کد زیاد و تکراری بود. همه‌ی این کارهارو بردم توی یه interface تا کار ساختن یه instance ساده تر انجام بشه.
دلیل اینکه توی یه فایل جدا نوشتم هم این بود چون خیلیا با ورژن قبلی راحت‌ترن، بعد آپدیت نیاز نباشه مجدد کدشون رو دست بزنند

@birddevelper
Copy link
Contributor

ممنون از مشارکت ارزشمند شما @mojtabaakbari221b
لطفا برنچ خودتون رو با اخرین تغییرات برنچ main ریپازیتوری اصلی ریبیس کنید تا امکان مرج کردن داشته باشیم.

callback_url=callback_url,
mobile_number=mobile_number,
)
return bank.redirect_gateway()
Copy link
Contributor

Choose a reason for hiding this comment

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

لطفا در مثالتون قبل از فراخوانی redirect_gateway متد ready رو هم اضافه کنید تا اگر کسی از مثال شما استفاده کرد به مشکل بر نخوره.

# ...
bank_record = bank.ready()
return bank.redirect_gateway()
# ...

@mojtabaakbari221b
Copy link
Contributor Author

تغییراتی که گفتید رو اعمال کردم. ممنونم از شما

@birddevelper
Copy link
Contributor

تشکر آقای اکبری عزیز. @mojtabaakbari221b

@ali-zahedi ali-zahedi merged commit 90d176d into ali-zahedi:main Oct 24, 2024
1 check failed
Copy link

🎉 This PR is included in version 2.0.12 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants