Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Oct 9, 2025. It is now read-only.

Use Autocomplete serialization group (gain -75% of number of SQL requests)#160

Merged
pamil merged 1 commit into
Sylius:masterfrom
jacquesbh:patch-1
Dec 8, 2020
Merged

Use Autocomplete serialization group (gain -75% of number of SQL requests)#160
pamil merged 1 commit into
Sylius:masterfrom
jacquesbh:patch-1

Conversation

@jacquesbh
Copy link
Copy Markdown
Contributor

@jacquesbh jacquesbh commented Nov 27, 2020

I have around 720 products.

And this little change did a big change in response time.

image

@jacquesbh jacquesbh requested a review from a team as a code owner November 27, 2020 14:52
@pamil pamil merged commit 62485d9 into Sylius:master Dec 8, 2020
@pamil
Copy link
Copy Markdown
Contributor

pamil commented Dec 8, 2020

Thank you, Jacques! 🎉

@lchrusciel lchrusciel mentioned this pull request Oct 27, 2021
Merged
Zales0123 added a commit that referenced this pull request Oct 28, 2021
@Zales0123
bug #177 [API] Add missing "Autocomplete" serialization group (lchrus…
1f5c604 
…ciel)

This PR was merged into the 1.1-dev branch.

Discussion
----------

We should add this serialization group to the second product variant endpoint as well. 

Ref. #160

Based on #179

Commits
-------

f26ce33 [API] Add missing "Autocomplete" serialization group
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jacquesbh @pamil