Skip to content

Commit 8e9fe15

Browse files
authored
Merge pull request #417 from judgej/master
Note on legacy use of complete* being replaced by acceptNotification()
2 parents e807257 + 467522d commit 8e9fe15

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

README.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -475,6 +475,12 @@ $notification->getMessage(); // Additional message, if any, provided by the gate
475475
// update the status of the corresponding transaction in your database
476476
```
477477

478+
**Note:** some earlier gateways used the `completeAuthorize` and `completePurchase` messages to handle the incoming
479+
notifications. These are being converted and the `complete*` messages deprecated.
480+
They won't be removed in OmniPay 2.x, but it is advisable to switch to the `acceptNotification` message when convenient.
481+
An example is Sage Pay Server [completeAuthorize](https://github.com/thephpleague/omnipay-sagepay/blob/master/src/ServerGateway.php#L81)
482+
which is now handled by [acceptNotification](https://github.com/thephpleague/omnipay-sagepay/blob/master/src/ServerGateway.php#L40).
483+
478484
## Example Application
479485

480486
An example application is provided in the [omnipay/example](https://github.com/thephpleague/omnipay-example) repo.

0 commit comments

Comments
 (0)