- Assigned to i-trokhanenko
- Issue was unassigned.
- πΊπ¦Ukraine i-trokhanenko Lutsk πΊπ¦
Here is a related issue on Github - https://github.com/yabacon/paystack-php/issues/36
- πΊπ¦Ukraine i-trokhanenko Lutsk πΊπ¦
As I see on GitHub the
yabacon/paystack-php
library used here is unsupported and outdated. I would suggest to completely rewrite the module according to the original Paystack API - https://paystack.com/docs/api/#introduction - π³π¬Nigeria chike Nigeria
Wao, complete rewrite? It can't work like a drop-in replacement of the library and updating some functions?
- πΊπΈUnited States DamienMcKenna NH, USA
Apparently the fix is in the "master" branch on the library's repository. In theory that should be possible to add to a project via this command:
composer require yabacon/paystack-php:dev-master
However, because this module requires v2.1 or newer, composer will likely complain that "dev-master" (i.e. the master branch) does not fulfill that requirement.
One approach would be to create a merge request here that changes the dependency to say this:
"yabacon/paystack-php": "dev-master"
- @damienmckenna opened merge request.
- Status changed to Needs review
over 1 year ago 7:34pm 1 March 2023 - πΊπΈUnited States DamienMcKenna NH, USA
I created a merge request with the proposed change, and here's a patch for good measure.
Note: in order to use this in Composer you'll need to update your site's composer.json to use the merge request directly, instead of the patch.
- πΊπΈUnited States DamienMcKenna NH, USA
This blog post explains how to use a merge request in your site's composer.json file: https://www.mediacurrent.com/blog/how-fix-catch-22-problem-drupal-9-fixe...
- π³π¬Nigeria chike Nigeria
MR3 worked when applied following the instructions at #10.
- πΊπΈUnited States DamienMcKenna NH, USA
FYI in the github issue I suggested the maintainers tag a new release, as that would solve everyone's problems, and you wouldn't need to change this module at all.