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

Sync mit upstream/master #14

Merged
merged 17 commits into from
Nov 9, 2023
Merged

Sync mit upstream/master #14

merged 17 commits into from
Nov 9, 2023

Conversation

BashPrince
Copy link

Merged Änderungen vom Upstream Ticketpark Branch in den master Branch unserer Fork. Da wir unsere eigenen Changes teilweise in unseren geforkten master gemerged haben bevor wir PRs zu Upstream Ticketpark erstellt haben (und diese PRs dann teilweise gesquasht wurden) waren gleiche Changes unter unterschiedlichen Commits in upstream/master und unserem master. Der resultierende Mergekonflikt wurde hier behoben indem einfach die upstream/master Variante akzeptiert wurde, die eh bereits alle unsere Änderungen enthält.

fabianerni and others added 17 commits April 13, 2023 16:39
* Implement Transaction Initialize including test and example

* Correct annotation in InitializeRequest.php

* Fix constructor codestyle InitializeRequest.php

* Fix Code Style with php-cs-fixer

* Add Transaction Authorize Request and Response

* Add Transaction Authorize test and example

* Change Line Endings

* Delete non-existing property RedirectUrl from InitializeResponse.php in Transaction Namespace

* Fix Annotations in AuthorizeResponse

---------

Co-authored-by: Jozo Dra <[email protected]>
Co-authored-by: jdraguljic <[email protected]>
Co-authored-by: sagyemang <[email protected]>
…ns_in_fraud_prevention

Add type annotations in FraudPrevention
…pstream

# Conflicts:
#	lib/SaferpayJson/Request/Transaction/AuthorizeRequest.php
#	lib/SaferpayJson/Request/Transaction/InitializeRequest.php
Copy link

@drjozo drjozo left a comment

Choose a reason for hiding this comment

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

Schaut gut aus! Thx

@drjozo drjozo merged commit 8a25185 into master Nov 9, 2023
16 checks passed
@drjozo drjozo deleted the wartung/sync_to_upstream branch November 9, 2023 17:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants