academe / OmniPay-Payone

A PAYONE gateway driver for the OmniPay framework
MIT License
15 stars 16 forks source link

Notification address details - prefix field names with "billing" #1

Closed judgej closed 7 years ago

judgej commented 8 years ago

The names and address parts need a "billing" prefix to make them more consistent with the names of the items that were originally sent. There are also "shipping" name and address details sent through the notification callback, so it will help to keep the two distinct from each other.

judgej commented 7 years ago

Since there are lots of installs now, this change either needs to be abandoned, deprecated (running both prefixed and non-prefixed at the same time) or switchable as an option (yuk).

judgej commented 7 years ago

We'll leave this for now, and bring it back if there is a demand.