Skip to content

Conversation

@AmirMehrabi
Copy link
Contributor

Hello.

Great package. But I needed to add metadata to my requests. This pull request adds the ability to just do that.

@matscode
Copy link
Owner

Nice initiative @AmirMehrabi .

However, this package as moved to Paystack PHP SDK and I am guessing it has the ability to add metadata already.

I recommened you use the new package as it is the only one that would receive possible future update.

@AmirMehrabi
Copy link
Contributor Author

Nice initiative @AmirMehrabi .

However, this package as moved to Paystack PHP SDK and I am guessing it has the ability to add metadata already.

I recommened you use the new package as it is the only one that would receive possible future update.

Hello,
I hope you're doing well.

I actually saw that package when I wanted to integrate paystack. But unfortunately, we're still on legacy code and for those of us who are still using old versions of Laravel, that "Paystack PHP SDK" package has unresolvabale dependencies so we can't use it.

But this one is still usable for old versions of laravel. That's why I used this one instead. And there are probably people like me out there, so merging this PR would help them as well.

Thank you,
Regards

@matscode
Copy link
Owner

Nice initiative @AmirMehrabi .
However, this package as moved to Paystack PHP SDK and I am guessing it has the ability to add metadata already.
I recommened you use the new package as it is the only one that would receive possible future update.

Hello, I hope you're doing well.

I actually saw that package when I wanted to integrate paystack. But unfortunately, we're still on legacy code and for those of us who are still using old versions of Laravel, that "Paystack PHP SDK" package has unresolvabale dependencies so we can't use it.

But this one is still usable for old versions of laravel. That's why I used this one instead. And there are probably people like me out there, so merging this PR would help them as well.

Thank you, Regards

I see your point... 🤚🏽

@matscode matscode merged commit 2c03c1a into matscode:master Jul 25, 2024
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.

2 participants