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

Kuveyt Türk API Versiyon Güncellemesi #172

Closed
FikretCin opened this issue Jan 5, 2024 · 5 comments · Fixed by #194
Closed

Kuveyt Türk API Versiyon Güncellemesi #172

FikretCin opened this issue Jan 5, 2024 · 5 comments · Fixed by #194

Comments

@FikretCin
Copy link
Contributor

Merhabalar;

Kuveyt POS kullanıcılarımıza bankadan gelen maile istinaden bildirmek istedim api v1 kullanımı ocak sonunda son bulacak v2 devreye alınacakmış. Paralel kullanım devam edecek mi bilmiyorum her ihtimale karşı v2 dökümanlarını paylaşmak istedim. Kodlamaya vakit bulup maalesef katkıda bulunamayacağım.

Non3D Model.pdf
3D Secure Model Kurulum Dokümanı.pdf
İptal_İade_Kısmiİade.pdf

Ayrıca şahsi not eklemek isterim 3Ds için zorunlu tuttukları alanlar (il isocode, ülke isocode vs.) gereksiz saçma ve pakedin order objesine müdahale gerektirecek şeyler.

Kolaylıklar dilerim.

@nuryagdym
Copy link
Collaborator

Merhabalar,
Bilgiler icin tesekkurler.
Evet bu alanlarin
image
zorunlu olmasi sacma olmus, ama zorunlu derken alanlara gercek deger vermeyi mi kastediyor acaba, yoksa bosh veya sabit degerler gonderebilir miyiz.

Kuveytin entegrasyonu zor. Yurt disindan (bulundum yer) API'larina istek gonderince cevap alamiyorum. Test ortamini yurt disi IP'lerine bloklamislar sanirim. Ayrica test API hesap bilgileri lazim olacak.

Benim de cok ta baslayasim yok buna :). v1'i bir yayina alayim sonra bakarim, yani Mart'tan once olmayabilir.
Cok bir degisiklik gerektirmiyorsa v1 ile beraber yayina alirim subata kadar

@nuryagdym
Copy link
Collaborator

Yeni aklima geldi, 15 sene once amerikadayken ben internetten alis veris yaparken Billing adresi zorunluydu, websitede girdigim billing address ile debit karti aldigim bankadaki kayitli adresim uyusmadigi zaman odeme basarisiz aliyordu. Ayni durum olabilir burda da.

@FikretCin
Copy link
Contributor Author

FikretCin commented Jan 5, 2024

Anladığım kadarıyla request bodyde olması zorunlu ve değerlerinin tespiti de zor örneğin :

<BillAddrCity>İstanbul</BillAddrCity>

<BillAddrCountry>792</BillAddrCountry> // Türkiye yerine ISO 3166-2 kodu

<BillAddrLine1>XXX Mahallesi XXX Caddesi No 55 Daire 1</BillAddrLine1>

<BillAddrPostCode>34000</BillAddrPostCode>

<BillAddrState>34</BillAddrState> İl ISO 3166-2 kodu

-- Ek olarak "Ayrica test API hesap bilgileri lazim olacak" her pdf in en alında test apileri ve test kartı olması lazım.

Edit: Test ortamında hardcode tutarsız veri göndersekte geçiyor (City İstanbul, State 40 gibi).

@mettleshade
Copy link

Merhabalar, sistemlerde bugun itibariyle kesinti yaşanacağı bildirildi ve şuanda bende iptal ve iadede sorun var, RetryCount alanını istiyor ama başka ne istiyor tam tespit edemedim. @nuryagdym @FikretCin

@nuryagdym
Copy link
Collaborator

gereken guncellemeleri yaptim, ama yurt dusunda oldugumdan ve kuveyt'in IP kisitlamasindan dolayi test edemiyorum.

composer require symfony/event-dispatcher mews/pos:dev-172-kuveyt-turk-api-versiyon-guncellemesi

ile bu branchteki degisiklikleri cekip test eder misiniz.

Not:

  1. Bu guncelleme v1.0.0'e eklendi, bu yuzde once kodlarinizi v1.0.0 ile calisacak sekilde guncellemeniz gerekiyor.
  2. Kuveyt'a ozel olarak ekstra alanlar istedigi icin bu alanlari kutuphane icine eklemedim, ama isteklerin calismasi icin bu alanlar gonderilmesi gerekiyor. Bunun yerine yeni eklenen event'larla bu ozel alanlari ekledim, ekstra alan ekleme ornegini burda bulablirsiniz

nuryagdym added a commit that referenced this issue Apr 23, 2024
…-guncellemesi

issue #172 - upgrade KuveytPos version to TDV2.0.0
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 a pull request may close this issue.

3 participants