Skip to content

Add same items as ordered#1210

Open
MaciekVoucherify wants to merge 3 commits into
masterfrom
MK-same-items
Open

Add same items as ordered#1210
MaciekVoucherify wants to merge 3 commits into
masterfrom
MK-same-items

Conversation

@MaciekVoucherify

Copy link
Copy Markdown
Collaborator

No description provided.

@MaciekVoucherify MaciekVoucherify self-assigned this Apr 14, 2026
@MaciekVoucherify MaciekVoucherify added Documentation Improvements or additions to the documentation DO NOT MERGE Documentation-team Managed by the Documentation team Articles Changes to articles or new article labels Apr 14, 2026
@mintlify

mintlify Bot commented Apr 14, 2026

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
voucherify 🟢 Ready View Preview Apr 14, 2026, 11:48 AM

@p-woznikowski p-woznikowski left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The changes to discount-types page are very confusing.

The product-bundles page can be improved.


### Add same items as ordered

This effect calculates the number of free units based on purchased items, but you must explicitly define where the free units should be applied.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Honestly, I have problems with understanding line.

...but you must explicitly define where the free units should be applied.

I don't understand this bit even though I've also checked this feature.

Please improve and use items in the cart, as purchased items sound more like the order's paid, and this is not the case yet.


This effect calculates the number of free units based on purchased items, but you must explicitly define where the free units should be applied.

The free items are applied only within the scope defined in **Discounted products**. They are not automatically assigned to the same products that triggered the discount.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is something that you've added on the basis of what's for BOGO, but this is confusing here. This is Rewards, so They are not automatically assigned to the same products that triggered the discount. is not really relevant.

Please redo the whole section as it doesn't describe what the Rewards > Free products: Add same items as ordered feature does.

Mention that it's primarily for adding products from a collection. Test it in the Demo shop and compare results in logs/activities of respective orders.

Comment thread documentation/build/product-bundles.mdx Outdated
Comment on lines +142 to +144
<Info>
Free items are not automatically assigned to the same products that triggered the bundle — they are applied only within the scope you define.
</Info>

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This repeats the content of the previous sentence, don't you think?

If necessary, rephrase both sentences. This callout is unnecessary.

- **Discount type:** Free products → Add same items as ordered
- **Free units:** 1
- **Discounted products:** Collection "T-Shirts"
- **Multiply discount by each bundle added**

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK, but one improvement; this presupposes that Maximum number of eligible bundles is 3 or more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Articles Changes to articles or new article DO NOT MERGE Documentation Improvements or additions to the documentation Documentation-team Managed by the Documentation team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants