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

V1.0.0 #192

Merged
merged 259 commits into from
Mar 30, 2024
Merged

V1.0.0 #192

merged 259 commits into from
Mar 30, 2024

Conversation

nuryagdym
Copy link
Collaborator

@nuryagdym nuryagdym commented Mar 23, 2024

@huseyinyildirim
@ilyasokay
@vedatyilmaz
@mahmut
@umuttaymaz
@zeydduran
@emreakay
@erenilhan
@FikretCin
@skuzoluk
@ibayazit
@sekizlipenguen

Kutuphanenin V1 versiyonu icin istedigim degisiklikleri tamamladim, yeni degisikliklerle ilgili dusuncelerinizi almak isterim.

Degisikliklerini ozetini refactoring-08 branch'de /docs/CHANGLE.LOG.md'den gorebilirsiniz.

/docs altinda kod ornekleri de bulabilirsiniz.

@sekizlipenguen
Copy link

öncelikle eline sağlık.

estpos yönteminde create3DPaymentRequestData fonksiyonuna extra data girişi yapmam gerekiyordu değişikliklerde bunu rastlayamadım bu yöntemi geçici olarak 16.0 versiyonunda şu şekilde çözmüştüm.
Screenshot 2024-03-25 at 23 16 21

@nuryagdym
Copy link
Collaborator Author

@sekizlipenguen extra bilgiyi response.php'de
su sekilde ekleyebilirsiniz:

    $eventDispatcher->addListener(RequestDataPreparedEvent::class, function (RequestDataPreparedEvent $event) {
        if ($event->getTxType() === PosInterface::TX_TYPE_PAY_AUTH) {
            $data                    = $event->getRequestData();
            $data['Extra'] = [
                //extra veriler
            ];
            $event->setRequestData($data);
        }
    });

ornekleri burda /docs'ta

ve examples 'larda bulabilirisiniz.
Bu tur degisiklikler icin artik kutuphan kodunu guncellemeye gerek yok.

@nuryagdym nuryagdym merged commit 129bf95 into master Mar 30, 2024
4 checks passed
@nuryagdym nuryagdym deleted the refactoring-08 branch August 20, 2024 09:40
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