Skip to content
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

Needs updated: Compatibility with algoliaSearch parameters #389

Closed
bidoubiwa opened this issue May 3, 2021 · 4 comments
Closed

Needs updated: Compatibility with algoliaSearch parameters #389

bidoubiwa opened this issue May 3, 2021 · 4 comments
Labels
documentation Improvements or additions to documentation

Comments

@bidoubiwa
Copy link
Contributor

bidoubiwa commented May 3, 2021

Compatibility

All bulletpoints that have been crossed are UI related and do not need to be tested.

Not compatible features list at the end of the issue.
Not compatible parameters start with a ❌.

search

  • query: The text to search in the index
    ❗️ Not tested because completely ignored by instantSearch

  • similarQuery: Overrides the query parameter and performs a more generic search that can be used to find “similar” results.
    ❗️ Not tested because usefulness is questionable

attributes

  • ✅ attributesToRetrieve: Gives control over which attributes to retrieve and which not to retrieve.

    • In MeiliSearch attributesToRetrieve does not impact attributeToHighlight
  • ❌ restrictSearchableAttributes: Restricts a given query to look in only a subset of your searchable attributes.
    This setting overrides searchableAttributes for specific searches, without changing the priority order of the searchableAttributes.

ranking

  • ❌ relevancyStrictness: Controls the relevancy threshold below which less relevant results aren’t included in the results.

filtering

  • ✅ filters: Filter the query with numeric, facet and/or tag filters. Add integration and unit tests on filters #393

    • ✅ 'attribute = value AND | OR | NOT attribute = value' // Different usage than instantSearch Docs
    • ✅ 'numeric_attribute = | != | > | >= | < | <= numeric_value'
    • ❌ 'attribute:lower_value TO higher_value'
    • ❌ 'facetName:facetValue'
    • ❌ '_tags:value'
    • ✅ 'attribute = value' // Different usage than instantSearch Docs
  • ✅ facetFilters: Filter hits by facet value. Facets tests #400

  • ❌ optionalFilters: Create filters for ranking purposes, where records that match the filter are ranked higher, or lower in the case of a negative optional filter.

  • ✅ numericFilters: Filter on numeric attributes. Add integration and unit tests on filters #393

  • ❌ tagFilters: Filter hits by tags.

  • ❌ sumOrFiltersScores: Determines how to calculate the total score for filtering.

faceting

  • ✅ facets: Facets to retrieve. Facets tests #400

  • ❌ maxValuesPerFacet: Maximum number of facet values to return for each facet during a regular search.
    If you want to change the number of retrieved facet hits during a search for facet values, see maxFacetHits.

this is possible using limit in refinement-list
Example:
<ais-refinement-list limit="3" attribute="boardgamecategory" />

  • ❌ facetingAfterDistinct: Force faceting to be applied after de-duplication (via the Distinct setting).

  • ❌ sortFacetValuesBy: Controls how facet values are sorted.

possible using transform Items on refinements

highlighting-snippeting

pagination

Not Compatible

See instantSearch documentation after Pagination on this page.

  • ❌ typos parameters
  • ❌ geo-search parameters
  • ❌ languages parameters
  • ❌ advanced parameters
bors bot added a commit that referenced this issue May 25, 2021
408: Add snippet/highlight/pagination integration tests r=bidoubiwa a=bidoubiwa

see #389 

Unit tests have also been added see: 
#387


Co-authored-by: Charlotte Vermandel <charlottevermandel@gmail.com>
@bidoubiwa bidoubiwa changed the title Add configuration tests Compatibility with algoliaSearch parameters Jun 1, 2021
@meck93
Copy link

meck93 commented Oct 20, 2021

Hi @bidoubiwa

Are there plans to support offset and limit in the future?

We had a similar idea as described in ticket #521.

  • We want to only fetch what is necessary i.e. what is displayed on the current page (e.g., determine by hitsPerPage). Only when the user clicks "Load more" or the next page (e.g., 2) in the pagination case, a new query should be dispatched fetching the next batch of documents. Similar to this example from Algolia: https://instantsearchjs.netlify.app/stories/?path=/story/results-infinitehits--default where "Load more" dispatches a new query. They use hitsPerPage and page to determine the correct batch of documents.
  • Alternatively, we had the idea to fetch the first 20 results and once required (e.g., when navigating to the 5th page if hitsPerPage=5) another search query would be dispatched.

Thanks for your work and feedback! Let me know if the comment was placed on the wrong issue. I'll happily move it elsewhere (or to the discussion).

@bidoubiwa
Copy link
Contributor Author

Hey @meck93 as you are suggesting a feature, an issue would have been more suited. But no worries, I opened one you can continue answering there: #569

@meck93
Copy link

meck93 commented Oct 25, 2021

hi @bidoubiwa thanks a lot for creating a new issue and for formulating it as a feature request. If you want to keep this issue clean, feel free to delete my comments.

@bidoubiwa bidoubiwa changed the title Compatibility with algoliaSearch parameters Needs updated: Compatibility with algoliaSearch parameters Apr 5, 2022
@bidoubiwa bidoubiwa added the documentation Improvements or additions to documentation label Sep 27, 2022
@bidoubiwa
Copy link
Contributor Author

Outdated and requires to much maintenance

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

2 participants