Skip to content

Revise product download documentation for clarity#1640

Open
Vidhan2104 wants to merge 17 commits intohotwax:user-guides-pubfrom
Vidhan2104:updated-product-sync
Open

Revise product download documentation for clarity#1640
Vidhan2104 wants to merge 17 commits intohotwax:user-guides-pubfrom
Vidhan2104:updated-product-sync

Conversation

@Vidhan2104
Copy link
Copy Markdown
Contributor

Updated the product download documentation to clarify the synchronization process and added details on managing newly added products.

Updated the product download documentation to clarify the synchronization process and added details on managing newly added products.
@Vidhan2104 Vidhan2104 requested a review from dt2patel April 30, 2026 06:27
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the Shopify product download documentation to describe a more detailed seven-stage synchronization process involving GraphQL Bulk Operations and SHA-256 hashing for change detection. It also removes the section on regular product imports. Feedback focuses on adhering to the HotWax style guide by removing marketing fluff (e.g., "efficient", "intelligent"), pruning unnecessary words (e.g., "blindly", "automatically"), avoiding banned terms like "ensure", and correcting punctuation in bullet points.

Comment thread documents/learn-shopify/shopify-integration/products/download-products.md Outdated
Comment thread documents/learn-shopify/shopify-integration/products/download-products.md Outdated
Comment thread documents/learn-shopify/shopify-integration/products/download-products.md Outdated
Comment thread documents/learn-shopify/shopify-integration/products/download-products.md Outdated
Comment thread documents/learn-shopify/shopify-integration/products/download-products.md Outdated
Comment thread documents/learn-shopify/shopify-integration/products/download-products.md Outdated
Comment thread documents/learn-shopify/shopify-integration/products/download-products.md Outdated
Comment thread documents/learn-shopify/shopify-integration/products/download-products.md Outdated
Vidhan2104 and others added 16 commits April 30, 2026 12:09
…products.md

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
…products.md

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
…products.md

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
…products.md

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
…products.md

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
…products.md

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
…products.md

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
…products.md

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Updated the explanation of product deletion processes to enhance clarity and maintain data integrity. Added details on soft deletion and how deletions are detected during synchronization.
Added a header and introductory content to the delete-products guide.
Clarified the preparation and processing steps for physical and digital gift cards. Updated the explanation of how HotWax Commerce downloads gift cards from Shopify.
Updated the description of how HotWax Commerce downloads kit products and their components, clarifying the integration with ERP.
Updated the description and improved clarity on how product updates are identified and synchronized between Shopify and HotWax Commerce.
Updated the description to clarify synchronization of product data across multiple Shopify stores. Revised sections on product synchronization and linking processes for better clarity.
Copy link
Copy Markdown
Contributor

@dt2patel dt2patel left a comment

Choose a reason for hiding this comment

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

This rewrite is not ready as-is. It changes established job names and behavior descriptions, and the new sync explanation includes at least one logic error (). Please reconcile the content with the current product-sync docs and job terminology before merge.

Copy link
Copy Markdown
Contributor

@dt2patel dt2patel left a comment

Choose a reason for hiding this comment

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

This rewrite is not ready as-is. It changes established job names and behavior descriptions, and the new sync explanation includes at least one logic error: If no hashes match, the system knows no changes occurred. Please reconcile the content with the current product-sync docs and job terminology before merge.

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