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

WIP: SS4 upgrade #111

Open
wants to merge 8 commits into
base: 2.1
Choose a base branch
from
Open

WIP: SS4 upgrade #111

wants to merge 8 commits into from

Conversation

vinstah
Copy link

@vinstah vinstah commented Oct 5, 2020

Hello,

I've started to update the code to be able to be used in SS4

clone repo into a new base install of SilverStripe 4, I'm not using composer to install as the payment module needs to be updated for SS4 as well.

@elliot-sawyer
Copy link

Hi there, thanks for the pull request! I believe the payments module used for this project has been abandoned. A better approach would be to refactor using https://github.com/silverstripe/silverstripe-omnipay/, which is already compatible with Silverstripe 4

@vinstah
Copy link
Author

vinstah commented Oct 6, 2020

Hi,
good point I'll look into omnipay,

I also saw that sunnysideup/ecommerce could be the successor to this module but for people wanting to upgrade their current platform to SS4 this update is in need.

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