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

Back in Stock email not triggered in Commerce 5 #35

Open
eeeeeeeeeve opened this issue Jan 3, 2025 · 1 comment
Open

Back in Stock email not triggered in Commerce 5 #35

eeeeeeeeeve opened this issue Jan 3, 2025 · 1 comment

Comments

@eeeeeeeeeve
Copy link

eeeeeeeeeve commented Jan 3, 2025

Describe the bug

In Commerce 5, inventory/stock levels are separate from the Variant element, so the event listener is never triggered.

Additionally, can the dashboard logs be improved so that the product link is to the Variant itself? It currently links to the product type.

Steps to reproduce

  1. In Craft 5/Commerce 5, set the stock to 0 and fill out frontend form
  2. Set stock to >0

Craft CMS version

5.5.8

Plugin version

4.0.5

Multi-site?

No

Additional context

No response

@engram-design
Copy link
Member

Great point, completely overlooked that when bumping it to Commerce 5. What's frustrating however is that there seems to be no inventory-based events available in Commerce 5 to hook into. I'll need to submit a PR to Commerce to add some events so that we can actually know about inventory changes. Otherwise, we're completely stuck!

The product/variant link is fixed for the next release. To get this early, run composer require verbb/back-in-stock:"dev-craft-5 as 4.0.5".

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

No branches or pull requests

2 participants