docs: clarify Shopify subscription signal update behavior#9
docs: clarify Shopify subscription signal update behavior#9lordhumunguz wants to merge 1 commit intomasterfrom
Conversation
📝 WalkthroughWalkthroughDocumentation updates describing changes to Shopify order-line classification signals, including new signal precedence logic (explicit one-time signals win), expanded metadata key recognition, placeholder value guardrails, and expected user impact regarding order reclassification. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~18 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
🧹 Nitpick comments (2)
data-transformations/order-segmentation/order-type.mdx (1)
33-58: Well-structured update section with clear user-facing expectations.A couple of minor observations:
"Post-cutover" jargon (line 44): Same as in the ReCharge FAQ — consider using "after the February 20, 2026 update" for consistency and reader clarity.
Lines 56–57 nicely set user expectations about potential reclassification in both directions. Line 58 reassuring no schema change is a good touch.
Optional: minor wording adjustment on line 44
-- Added Shopify app-id fallback coverage for single-line orders: - - Includes ReCharge app id `294517` in the post-cutover order-source fallback list. +- Added Shopify app-id fallback coverage for single-line orders: + - Includes ReCharge app id `294517` in the order-source fallback list (effective February 20, 2026).🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@data-transformations/order-segmentation/order-type.mdx` around lines 33 - 58, The phrase "post-cutover" in the Shopify update section is jargon; replace that wording with "after the February 20, 2026 update" for clarity and consistency with the ReCharge FAQ across the document (specifically update the phrase in the sentence that references the order-source fallback list), leaving the rest of the Shopify Metadata Update text (signal precedence, expectations, and schema reassurance) unchanged.help-center/faq/data-faqs/why-doesnt-recharge-match-sourcemedium.mdx (1)
47-53: Consider replacing "post-cutover" with a more reader-friendly phrase.Line 53 uses "post-cutover Shopify data" which is internal jargon. Since this is a user-facing FAQ, a phrase like "after the February 20, 2026 update" would be more immediately clear to customers who may not know what "cutover" refers to.
Suggested change
-- In post-cutover Shopify data, single-line orders with `order_source_name = '294517'` can now classify as subscription via fallback logic. +- After the February 20, 2026 update, single-line Shopify orders with `order_source_name = '294517'` can now classify as subscription via fallback logic.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@help-center/faq/data-faqs/why-doesnt-recharge-match-sourcemedium.mdx` around lines 47 - 53, Replace the internal jargon "post-cutover Shopify data" in the "Shopify metadata signal changes (for existing users)" section with a user-friendly phrase such as "after the February 20, 2026 update" (or the actual public-facing update date), updating the sentence "In post-cutover Shopify data, single-line orders..." to read e.g. "In Shopify data after the February 20, 2026 update, single-line orders..." so customers immediately understand the timing; make the same replacement for any other occurrences of "post-cutover" in this FAQ section.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@data-transformations/order-segmentation/order-type.mdx`:
- Around line 33-58: The phrase "post-cutover" in the Shopify update section is
jargon; replace that wording with "after the February 20, 2026 update" for
clarity and consistency with the ReCharge FAQ across the document (specifically
update the phrase in the sentence that references the order-source fallback
list), leaving the rest of the Shopify Metadata Update text (signal precedence,
expectations, and schema reassurance) unchanged.
In `@help-center/faq/data-faqs/why-doesnt-recharge-match-sourcemedium.mdx`:
- Around line 47-53: Replace the internal jargon "post-cutover Shopify data" in
the "Shopify metadata signal changes (for existing users)" section with a
user-friendly phrase such as "after the February 20, 2026 update" (or the actual
public-facing update date), updating the sentence "In post-cutover Shopify data,
single-line orders..." to read e.g. "In Shopify data after the February 20, 2026
update, single-line orders..." so customers immediately understand the timing;
make the same replacement for any other occurrences of "post-cutover" in this
FAQ section.
Summary
Updated pages
Summary by CodeRabbit