Skip to content

Add images connection field to Variant Type#29

Merged
ChristianRimondi merged 3 commits intomasterfrom
add-image-connection-to-product
Nov 29, 2019
Merged

Add images connection field to Variant Type#29
ChristianRimondi merged 3 commits intomasterfrom
add-image-connection-to-product

Conversation

@AlessioRocco
Copy link
Copy Markdown
Contributor

@AlessioRocco AlessioRocco commented Oct 18, 2019

It adds the images connection field to the Variant Type and uses the SolidusGraphqlApi::Queries::Variant::ImagesQuery query object with batch loader to retrieve the images.

@AlessioRocco AlessioRocco self-assigned this Oct 18, 2019
@AlessioRocco AlessioRocco force-pushed the add-image-connection-to-product branch from 8a02216 to 6fe55c5 Compare October 18, 2019 15:05
@AlessioRocco AlessioRocco force-pushed the add-image-connection-to-product branch from 6fe55c5 to 6517db2 Compare November 8, 2019 17:02
@AlessioRocco AlessioRocco force-pushed the add-image-connection-to-product branch 7 times, most recently from 173a001 to 476c0cf Compare November 22, 2019 18:11
It has basic informations and an urls for the styles provided by solidus.
@AlessioRocco AlessioRocco force-pushed the add-image-connection-to-product branch from 476c0cf to 6323611 Compare November 22, 2019 18:23
It adds the images connection field to the Variant Type and uses the
SolidusGraphqlApi::Queries::Variant::ImagesQuery query object with
batch loader to retrieve the images.
Using command "bundle exec rake schema:idl"
@AlessioRocco AlessioRocco force-pushed the add-image-connection-to-product branch from 6323611 to b4d6d24 Compare November 22, 2019 18:37
Copy link
Copy Markdown
Contributor

@ChristianRimondi ChristianRimondi left a comment

Choose a reason for hiding this comment

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

👍

@ChristianRimondi ChristianRimondi merged commit d0f8187 into master Nov 29, 2019
@ChristianRimondi ChristianRimondi deleted the add-image-connection-to-product branch November 29, 2019 09:06
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