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

Akbank (Payten) parametre talebine istinaden 'refreshtime' parametresi eklendi. #168

Merged
merged 5 commits into from
Dec 14, 2023

Conversation

FikretCin
Copy link
Contributor

CallbackUrl parametresi eklenmiş olmasına rağmen 3D ekranından siteye dönemeyen müşteriler için Payten destek ekibine tekrar başvurduk bize aşağıdaki mesajı ilettiler.

Merhaba Fikret Bey,

Callback karşılamak için şu kısma dikkat etmenizi rica ederim.

İsteklerinizde Refreshtime parametresini kullanmadığından burada default değer olan "300" saniye değerini baz alacaktır. Aşağıda ilettiğim refreshtime parametresini 10 saniye olarak iletmenizi rica ederim.

@nuryagdym
Copy link
Collaborator

nuryagdym commented Dec 8, 2023

Merhabalar, PR icin tesekkurler, refreshTime'in tam olarak ne ise yaradigini anlamamiz gerekiyor.
Dokumantasyonda bu alan hakkinda su aciklamayi buldum:

saniyeler içinde yönlendirilmeyi sağlayan sayaç değeri (okUrl veya failUrl' e yönlendirme süresi)

Ama yine de bu degeri neye gore kara verilecegini bilmemiz gerekiyor. Eger kullanicinin ihtiyacina gore degisecek bir deger ise hard coded 10 saniye yapmak yerine $order uzerinden gondermemiz gerekebilir.

@FikretCin
Copy link
Contributor Author

Destek ekibine tam olarak işlevini ve belirtilen değerin amacını sordum geri bildirimlerini sizinle paylaşacağım.

@FikretCin
Copy link
Contributor Author

Payten ekibi ile görüşme sağladım. Parametrenin sadece akbank değil tüm est altyapısında kullanılabileceğini işlemin ardından hata payı kalmaması için yönlendirme süresini kısmak için kullanıldığını ilettiler. Birim saniye cinsinden çalışmaktaymış tüm entegrasyonlara 10 saniyeyi öneriyoruz isterseniz 1 bile yapabilirsiniz dediler.
Örneğin kullanıcı OTP ekranında başarılı sonuç aldı fakat yönlendirmeyi beklemeden ekranı terk etti gibi durumlar için hem terk süresini azaltmak hemden terkin ardından 10 saniye içerisinde callbacke istek atmak için bu parametreyi kullanmaktalarmış.

@nuryagdym
Copy link
Collaborator

Tmm tesekkurler, ben bugun aksam farkli refreshtime degerler ile test edip sonra merge ederim.

@FikretCin
Copy link
Contributor Author

Ben teşekkür ederim release hem ziraat hem bunun için 4 gözle beklemekteyim 🎉

Copy link
Collaborator

@nuryagdym nuryagdym left a comment

Choose a reason for hiding this comment

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

Biraz deneme yaptim refreshTime => 10 ile.
Taraycida 3D onay sayfada onayladim ve websiteye geri yonlendirilmeden once kapattim tarayci sekmesini ama bir sonuc alamadim,

bu log kodu calismadi, yani callbackUrl'a bir istek gelmedi
https://github.com/mewebstudio/pos/blob/master/examples/_templates/_payment_response.php#L16-L20

@@ -270,6 +271,7 @@ public function create3DFormDataCommon(AbstractPosAccount $account, $order, stri
'failUrl' => $order->fail_url,
// musteri 3D onay sayfasinda islemleri tamamlamadan kapatirsa, banka bu sayfaya bir istek gonderecek.
'callbackUrl' => $order->fail_url,
'refreshtime' => '10', //callbackUrl sisteminin doğru çalışması için eklenmesi gereken parametre
Copy link
Collaborator

Choose a reason for hiding this comment

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

dokumantasyonda refreshTime olarak geciyor

@FikretCin
Copy link
Contributor Author

Callback live bir url mi localhost üzerinden mi çalışıyorsun ?

@nuryagdym
Copy link
Collaborator

evet localhost'ta serve ediliyor proje, callbackUrl'a verilen url'a bir istek gondermiyor

@FikretCin
Copy link
Contributor Author

callback live bir endpoint olması gerekiyor data alabilmen için ben expose kullanmıştım ngrok ta kullanabilirsin.

@nuryagdym
Copy link
Collaborator

o zaman reshrestime'i reshreshTime olarak duzeltir misin

@FikretCin
Copy link
Contributor Author

Release i bu gün çıkabilicek miyiz ?

@nuryagdym nuryagdym merged commit 5ff8f20 into mewebstudio:master Dec 14, 2023
2 checks passed
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.

2 participants