Skip to content
This repository was archived by the owner on May 28, 2023. It is now read-only.

Conversation

@gibkigonzo
Copy link
Contributor

@gibkigonzo gibkigonzo commented Jan 28, 2020

related: vuestorefront/vue-storefront#4001 vuestorefront/mage2vuestorefront#99

Description:
In general this change allow to fetch attributes with needed options
for example: {'color': [49]} => will return color attribute with option that has id 49
It's used only for products (and response is transformed into attribute_metadata), but we can expose it as endpoint if needed.

@gibkigonzo gibkigonzo changed the title [WIP] fetch attributes based on aggregates fetch attributes based on aggregates Jan 30, 2020
Copy link
Contributor

@pkarw pkarw left a comment

Choose a reason for hiding this comment

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

great job. we need to add support for this feature for the graphql resolvers as well

Copy link
Contributor

@andrzejewsky andrzejewsky left a comment

Choose a reason for hiding this comment

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

@gibkigonzo this is exactly what I meant! amazing work! 🎉

@gibkigonzo gibkigonzo requested a review from pkarw January 30, 2020 10:30
@andrzejewsky andrzejewsky merged commit 05998f8 into vuestorefront:develop Feb 11, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants