Skip to content

Fix update variation properties - #98

Merged
kengallego merged 4 commits into
ePages-de:wip-v2from
kengallego:fix-update-variation-properties
Jul 23, 2025
Merged

Fix update variation properties#98
kengallego merged 4 commits into
ePages-de:wip-v2from
kengallego:fix-update-variation-properties

Conversation

@kengallego

Copy link
Copy Markdown
Contributor

No description provided.

CopilotAI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes the handling of variation properties by updating the data structure and processing methods. The changes ensure that array-type variation properties are properly camelized and processed when making API calls.

  • Enhanced camelize_keys utility to handle arrays recursively, not just hashes
  • Updated update_variation_properties method documentation to reflect array parameter type
  • Removed redundant patch method from connection concerns

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

FileDescription
lib/beyond_api/utils.rbEnhanced camelize_keys to recursively process arrays in addition to hashes
lib/beyond_api/services/product_management/product.rbUpdated parameter documentation and example for variation properties
lib/beyond_api/concerns/connection.rbRemoved duplicate patch method implementation

Comment threadlib/beyond_api/utils.rb Outdated
Comment threadlib/beyond_api/services/product_management/product.rb
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@kengallego
kengallego merged commit 7d41026 into ePages-de:wip-v2Jul 23, 2025
@kengallego
kengallego deleted the fix-update-variation-properties branch July 23, 2025 06:11
Sign up for freeto 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

@kengallego