boomerdigital / solidus_flexi_variants

BSD 3-Clause "New" or "Revised" License
15 stars 31 forks source link

Update solidus compatibility #22

Open fastjames opened 4 years ago

fastjames commented 4 years ago

Update the CI testing matrix so we cover Solidus versions currently under maintenance.

I tried this once and I think I tried to cover too many Solidus versions in one go. This time I'm going to take smaller steps.

fastjames commented 4 years ago

Look at that, finally passing on all versions! I'm happy to squash it before merge.

fastjames commented 4 years ago

I have updated this PR to include v2.10.

afdev82 commented 4 years ago

any update on this?

dhonig commented 4 years ago

We can work on getting this merged if you are interested, you can also send a PR?

On Mon, Jun 15, 2020 at 4:38 PM Antonio Facciolo notifications@github.com wrote:

any update on this?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/boomerdigital/solidus_flexi_variants/pull/22#issuecomment-644375917, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABZTL2WRJ2KRKVO5R4RXXTRW2BD5ANCNFSM4JFFAFXA .

fastjames commented 4 years ago

I have not looked at this in several months. Is it missing a new version?

afdev82 commented 4 years ago

Yes, maybe I would need to use this extension for one project, but I'm not sure yet. In that case I would be happy to contribute, but I asked to know the current status of this extension. I used some extensions in the past and they needed refactoring to use the latest solidus because they were quite abandoned.

We can work on getting this merged if you are interested, you can also send a PR? On Mon, Jun 15, 2020 at 4:38 PM Antonio Facciolo @.***> wrote: any update on this? — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#22 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABZTL2WRJ2KRKVO5R4RXXTRW2BD5ANCNFSM4JFFAFXA .