Skip to content

Comments

Fix added to configurable products to pull child info and feature to generate child_info#40

Merged
tejasklevu merged 15 commits intomasterfrom
fix-configurable-child-info
Dec 2, 2025
Merged

Fix added to configurable products to pull child info and feature to generate child_info#40
tejasklevu merged 15 commits intomasterfrom
fix-configurable-child-info

Conversation

@sandipklevu
Copy link
Contributor

@sandipklevu sandipklevu commented Sep 9, 2025

Reference PRs.

  1. Child info generation of Configurable product #37 (comment)
  2. Configurable child Product issue #35 (comment)

nileshdbabar2023 and others added 11 commits September 2, 2025 18:52
- Fix added to configurable products to pull the child data
- Review changes updated
- Review changes updated
- php7.1 Added
- php7.2 Added
- php7.1 Added
- php7.2 Added
Child info generation of Configurable product
- child_info added in fieldsProvider for CSV file format
- Fix result array while imploding with multivaluedelimiter in CSV file generation
Copy link
Contributor

@nileshdbabar2023 nileshdbabar2023 left a comment

Choose a reason for hiding this comment

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

Changes looks , good

Copy link
Contributor

@tejasklevu tejasklevu left a comment

Choose a reason for hiding this comment

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

All good.

@sandipklevu
Copy link
Contributor Author

@nileshdbabar2023 When you get chance, could you review the updated changes once. thanks

Copy link
Contributor

@tejasklevu tejasklevu left a comment

Choose a reason for hiding this comment

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

All Good from my side.

Copy link
Contributor

@nileshdbabar2023 nileshdbabar2023 left a comment

Choose a reason for hiding this comment

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

Changes looks good, can be merged

@tejasklevu tejasklevu merged commit b4748f3 into master Dec 2, 2025
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.

4 participants