Skip to content

bump min version of solidus_core dependency#176

Merged
elia merged 1 commit into
solidusio:masterfrom
loicginoux:bump-min-solidus-core-dependency
Feb 27, 2023
Merged

bump min version of solidus_core dependency#176
elia merged 1 commit into
solidusio:masterfrom
loicginoux:bump-min-solidus-core-dependency

Conversation

@loicginoux

@loicginoux loicginoux commented Feb 21, 2023

Copy link
Copy Markdown
Contributor

Summary

soldius Stripe makes use of Omnes gem and the Spree::Bus, both introduced in solidus 3.2.0.
When trying to install solidus_stripe v5 With a version of solidus_core below 3.2, Omnes gem and Spree::Bus are not present but the current minimum version of solidus core for the solidus_stripe gem is 2.3.
I bumped the minimal required version of solidus core.

Checklist

Check out our PR guidelines for more details.

The following are mandatory for all PRs:

The following are not always needed:

  • 📖 I have updated the README to account for my changes.
  • 📑 I have documented new code with YARD.
  • 🛣️ I have opened a PR to update the guides.
  • ✅ I have added automated tests to cover my changes.
  • 📸 I have attached screenshots to demo visual changes.

soldius Stripe makes use of Omnes and the Spree::Bus, both introduced in solidus 3.2.0.
With a version of solidus_core below 3.2, an app using this gem will not work
@loicginoux loicginoux mentioned this pull request Feb 21, 2023
3 tasks
@elia elia added this to the v5 milestone Feb 22, 2023
@elia elia merged commit a7e4621 into solidusio:master Feb 27, 2023
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