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

Upgrate to Spree 4.7 #968

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Upgrate to Spree 4.7 #968

wants to merge 2 commits into from

Conversation

theachoem
Copy link
Collaborator

@theachoem theachoem commented Jan 8, 2024

Changes

BIG change of spree 4.6 is translation. It only has a few updates that break the current 4.5 that we are using.

Task breakdown:

  • Upgrade to ~> 4.6.0 which mean >= 4.6 < 4.7 in spree_cm_commissioner.gemspec
  • Check new features
    1. Translations: new translation available for following models:
      Spree::Product, Spree::Taxon, Spree::Taxonomy, Spree::Property, Spree::ProductProperty, Spree::OptionType, Spree::OptionValue, Spree::Store
    2. New webhook subscriber secret_key
    • In 4.6, it introduce secret_key for webhook subscriber.
      • secret_key will be used to generate signature for webhook payload
      • which then set to headers['X-Spree-Hmac-SHA256'] when send the event to the subscriber.
  • Check if UI breaks
  • Make sure all spec/tests passed

Demo

References

@theachoem theachoem changed the title Spree 4.7 Upgrate to Spree 4.7 Jan 8, 2024
This was linked to issues Jan 8, 2024
@theachoem theachoem requested a review from channainfo January 8, 2024 03:39
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.

Upgrade to Spree 4.7 Upgrade to Spree 4.6
1 participant