Skip to content

Add code samples for sorting/filtering nested fields #2050

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Dec 22, 2022

Conversation

guimachiavelli
Copy link
Member

@guimachiavelli guimachiavelli commented Dec 15, 2022

Closes #2035

Code samples are currently in code blocks to ease reviewing, but will be moved to .code-samples.meilisearch.yaml and sample-template.yaml after approval.

Copy link
Contributor

@maryamsulemani97 maryamsulemani97 left a comment

Choose a reason for hiding this comment

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

Wouldn't it be better to have this in the filtering and sorting guides instead? 🤔

Copy link
Contributor

@maryamsulemani97 maryamsulemani97 left a comment

Choose a reason for hiding this comment

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

🍰

oluademola
oluademola previously approved these changes Dec 22, 2022
Copy link
Contributor

@oluademola oluademola left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks again @guimachiavelli 🎉

@maryamsulemani97
Copy link
Contributor

bors merge

bors bot added a commit that referenced this pull request Dec 22, 2022
2050: Add code samples for sorting/filtering nested fields r=maryamsulemani97 a=guimachiavelli

Closes #2035

Code samples are currently in code blocks to **_ease_** reviewing, but will be moved to `.code-samples.meilisearch.yaml` and `sample-template.yaml` after approval.

Co-authored-by: gui machiavelli <[email protected]>
@maryamsulemani97
Copy link
Contributor

bors r-

@bors
Copy link
Contributor

bors bot commented Dec 22, 2022

Canceled.

@maryamsulemani97 maryamsulemani97 removed their assignment Dec 22, 2022
Copy link
Contributor

@maryamsulemani97 maryamsulemani97 left a comment

Choose a reason for hiding this comment

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

You need to add the code samples to .code-samples.meilisearch.yaml and sample-template.yaml 👀

Copy link
Contributor

@maryamsulemani97 maryamsulemani97 left a comment

Choose a reason for hiding this comment

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

🍪

bors merge

@bors
Copy link
Contributor

bors bot commented Dec 22, 2022

@bors bors bot merged commit fc03ae7 into main Dec 22, 2022
@bors bors bot deleted the 2022-12-nested-fields-filter-sort branch December 22, 2022 13:58
bors bot added a commit that referenced this pull request Feb 16, 2023
2061: review filtering guide r=maryamsulemani97 a=maryamsulemani97

closes #1902, #860, #837, and #1850

- moves all operators to a new heading
- removes SQL syntax and combines it with the curl code samples
- replaces amazon and imdb images for facets with meilisearch demo images
- removes unused images in `.vuepress/public/faceted-search`
- remove redundant code samples
<hr>

- Improve coherence between dataset and examples
  - A sample document was added as part of  #2050 with `rating` and `director` fields
  - This PR uses `movie_ratings` instead of `movies` to clarify that this is a separate dataset
     - `faceted_search_facets_1` still  uses `movies` as this code sample is used in the API reference as well

Co-authored-by: Maryam Sulemani <[email protected]>
Co-authored-by: Maryam <[email protected]>
Co-authored-by: maryamsulemani97 <[email protected]>
bors bot added a commit that referenced this pull request Feb 16, 2023
2061: review filtering guide r=maryamsulemani97 a=maryamsulemani97

closes #1902, #860, #837, and #1850

- moves all operators to a new heading
- removes SQL syntax and combines it with the curl code samples
- replaces amazon and imdb images for facets with meilisearch demo images
- removes unused images in `.vuepress/public/faceted-search`
- remove redundant code samples
<hr>

- Improve coherence between dataset and examples
  - A sample document was added as part of  #2050 with `rating` and `director` fields
  - This PR uses `movie_ratings` instead of `movies` to clarify that this is a separate dataset
     - `faceted_search_facets_1` still  uses `movies` as this code sample is used in the API reference as well

Co-authored-by: Maryam Sulemani <[email protected]>
Co-authored-by: Maryam <[email protected]>
Co-authored-by: maryamsulemani97 <[email protected]>
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.

Include examples for filtering and sorting on a nested field
3 participants