Skip to content

Conversation

@GTamilSelvan07
Copy link
Contributor

Description

  • Purpose: Adds Shopify Storefront API integration for customer-facing e-commerce functionality, including product browsing, cart, collections, and customer account operations.
  • Approach: Implements the shopify-storefront integration with configuration, GraphQL queries/mutations, requirements, icon, docs, and tests for core Storefront scenarios.
    Type of change
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Updates

👉 Added shopify-storefront/ integration folder with implementation and config
👉 Includes product and collection queries, cart and customer mutations
👉 Documentation and test suite for all Storefront API scenarios

Screenshots

{Image here of before and after - if applicable}

Test plan

Provide guidance for how to QA your proposed changes. This is not only for a test but also useful for a reviewer.
{A test plan outlining scenarios to test}

Author(s) to check

  • Project and all contained modules builds successfully
  • Self-/dev-tested
  • Unit/UI/Automation/Integration tests provided where applicable
  • Code is written to standards
  • Appropriate documentation written (code comments, internal docs)

Copy link
Contributor

Choose a reason for hiding this comment

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

Is this necessary?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

oops sorry that fro internal docs so ican refer back ill remove them :)

Copy link
Contributor

Choose a reason for hiding this comment

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

Same with the whole folder, there should only be a README.md I would think

Added a comprehensive README.md to the shopify-storefront directory, consolidating integration usage, actions, and setup instructions. Removed outdated documentation files from the docs/ directory to streamline and centralize developer guidance.
Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Missing the SDK. How is this currently working without it?

@TheRealAgentK
Copy link
Collaborator

Just noticed that it also needs a README entry on the main level.

Added Shopify Storefront section to README with details on API capabilities.
@sumitramanga
Copy link
Contributor

Merge in next year - The dynamic URL needs to be merged in first. Refer to this PR here - https://github.com/Autohive-AI/Autohive/pull/2962

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.

3 participants