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.

Sign up for GitHub

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

Jump to bottom

PR -- [BUG] AdditionalData lacks the field paymentLinkId #788

Merged
merged 2 commits into from Oct 21, 2021
Merged

PR -- [BUG] AdditionalData lacks the field paymentLinkId #788

merged 2 commits into from Oct 21, 2021

Conversation

SergioBergaminiCodeploy
Copy link
Contributor

Description
This is a simple fix to issue #787. I also extended the notification test to be sure the paymentLinkId is actually deserialized.

@wboereboom
Copy link
Contributor

Can you please update this PR? We will include it in our next release!

@SergioBergaminiCodeploy
Copy link
Contributor Author

Hi @wboereboom !! If what you meant was to sync this PR with the last change in your dev branch, It should be okay now !!

@wboereboom wboereboom merged commit 932b5e6 into Adyen:develop Oct 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@maassenbas maassenbas maassenbas approved these changes

@wboereboom wboereboom wboereboom approved these changes

@AlexandrosMor AlexandrosMor Awaiting requested review from AlexandrosMor

@peterojo peterojo Awaiting requested review from peterojo

@zaiddreakh zaiddreakh Awaiting requested review from zaiddreakh

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@SergioBergaminiCodeploy @wboereboom @maassenbas