Skip to content

Conversation

@htdat
Copy link
Contributor

@htdat htdat commented Oct 31, 2025

TODOS:

  • Make sure that only two product types variable and simple are handled. We can have a config for this after merging Allow integrations to have custom parameters for queries #26. But I ideally, we would use wc_get_products with arg includes to get the single product we're want to validate.
  • All other product types are simply excluded. And give the message that: they're not supported.
  • For variable product, default is to display the first variation, but give the option to display multiple ones; for example, displaying links with variation product_id and then use it to parse the display.

Description

Testing instructions

Checklist

  • npm run test:php does not return errors.
  • npm run lint:php does not return errors.

@coderabbitai
Copy link

coderabbitai bot commented Oct 31, 2025

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch update/devhelper-openai-handle-vairations

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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