Skip to content

New version #11

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

Merged
merged 1 commit into from
May 20, 2025
Merged

New version #11

merged 1 commit into from
May 20, 2025

Conversation

leith
Copy link
Member

@leith leith commented Apr 7, 2025

  • add support for direct payments/MOTO
  • add unit tests to get coverage back to 100%
  • add explicit PHP 7.2+ requirement to composer
  • add different PHPUnit config for older PHP
  • change KOUrl parameter to use Omnipay's cancelUrl
  • change to PatronBase forks of common & tests
  • change from Travis to Github Actions
  • update composer dependencies
  • update PHPUnit config to mimic omnipay-common
  • fix failing unit test
  • fix up whitespace/phpcs messages
  • remove redundant notificationUrl and browserIp params (notifyUrl and clientIp already exist)

@leith leith force-pushed the new-version branch 2 times, most recently from 4c6fdcc to 8598f88 Compare April 7, 2025 05:12
* add support for direct payments/MOTO
* add unit tests to get coverage back to 100%
* add explicit PHP 7.2+ requirement to composer
* add different PHPUnit config for older PHP
* change KOUrl parameter to use Omnipay's cancelUrl
* change to PatronBase forks of common & tests
* change from Travis to Github Actions
* update composer dependencies
* update PHPUnit config to mimic omnipay-common
* fix failing unit test
* fix up whitespace/phpcs messages
* remove redundant notificationUrl and browserIp params
@leith leith marked this pull request as ready for review April 25, 2025 01:55
@leith leith merged commit 4ef8b92 into master May 20, 2025
10 checks passed
@leith leith deleted the new-version branch May 20, 2025 06:10
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.

1 participant