Skip to content
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

feat: support symfony 7 #555

Merged
merged 4 commits into from
Apr 3, 2024

Conversation

Chris53897
Copy link
Contributor

#552

Except from "symfony/dependency-injection": "^4.4 || ^5.4 || ^6.4",
This needs more work.

omnipay-common does not yet support symfony 7.
thephpleague/omnipay-common#264

@cyraid
Copy link

cyraid commented Feb 19, 2024

Now that you updated Omnipay, how does it look for some Symfony 7 support? :)

@Chris53897
Copy link
Contributor Author

Chris53897 commented Feb 19, 2024

It did a copy of the branch and it did work in production.
https://github.com/Chris53897/PayumBundle/tree/feature/support-symfony-7-srt

To make it stable for the ci, and safe to merge, it would be great to have a new release of https://github.com/thephpleague/omnipay-common
Update: Release is now available

@Chris53897
Copy link
Contributor Author

Now there is an installable set in the CI.

@pierredup It would be great if you can have a look and tell me which steps (and order) should we take to have full symfony 7 support.

@pierredup pierredup added this to the next milestone Apr 3, 2024
@pierredup
Copy link
Member

Thanks @Chris53897!

@pierredup pierredup merged commit 5bd3e1b into Payum:master Apr 3, 2024
12 checks passed
@Chris53897 Chris53897 deleted the feature/support-symfony-7 branch April 3, 2024 13:07
@CoderMaggie CoderMaggie mentioned this pull request Aug 23, 2024
74 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants