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.

Sign up for GitHub

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

Jump to bottom

When searching and using the matching strategy with a wrong value type we say « was expecting an object » while we’re actually expecting a string, this can be very confusing. #3391

Closed
Tracked by #3352
ManyTheFish opened this issue Jan 19, 2023 · 1 comment · Fixed by #3404
Closed
Tracked by #3352

When searching and using the matching strategy with a wrong value type we say « was expecting an object » while we’re actually expecting a string, this can be very confusing. #3391

ManyTheFish opened this issue Jan 19, 2023 · 1 comment · Fixed by #3404
Assignees
@ManyTheFish
Labels
bug Something isn't working as expected error handler Issues related to the returned errors in Meilisearch v1.0.0 PRs/issues solved in v1.0.0 released on 2023-02-06
Milestone
v1.0.0

Comments

@ManyTheFish
Copy link
Member

No description provided.

@ManyTheFish ManyTheFish mentioned this issue Jan 19, 2023
Closed
22 tasks
@ManyTheFish ManyTheFish self-assigned this Jan 19, 2023
@curquiza curquiza added this to the v1.0.0 milestone Jan 19, 2023
@curquiza curquiza added bug Something isn't working as expected error handler Issues related to the returned errors in Meilisearch labels Jan 19, 2023
This was referenced Jan 19, 2023
Merged
Merged
bors bot added a commit to meilisearch/milli that referenced this issue Jan 19, 2023
@bors @ManyTheFish
Merge #770
1c4b1b3 
770: Update deserr v0.3.0 r=irevoire a=ManyTheFish

related to meilisearch/meilisearch#3391


Co-authored-by: Many the fish <many@meilisearch.com>
bors bot added a commit that referenced this issue Jan 19, 2023
@bors @ManyTheFish
Merge #3404
913b933 
3404: Fix matching strategy error r=irevoire a=ManyTheFish

# Pull Request

## Related issue
Fixes #3391


Co-authored-by: ManyTheFish <many@meilisearch.com>
bors bot added a commit that referenced this issue Jan 19, 2023
@bors @ManyTheFish @irevoire
Merge #3404
49a205b 
3404: Fix matching strategy error r=irevoire a=ManyTheFish

# Pull Request

## Related issue
Fixes #3391


Co-authored-by: ManyTheFish <many@meilisearch.com>
Co-authored-by: Tamo <tamo@meilisearch.com>
@curquiza curquiza linked a pull request Jan 23, 2023 that will close this issue
Fix matching strategy error #3404
Merged
bors bot added a commit that referenced this issue Jan 23, 2023
@bors @ManyTheFish @dureuill
Merge #3404
fea41ca 
3404: Fix matching strategy error r=irevoire a=ManyTheFish

# Pull Request

## Related issue
Fixes #3391


Co-authored-by: ManyTheFish <many@meilisearch.com>
Co-authored-by: Louis Dureuil <louis@meilisearch.com>
@curquiza
Copy link
Member

Closed by #3404

@meili-bot meili-bot added the v1.0.0 PRs/issues solved in v1.0.0 released on 2023-02-06 label Feb 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ManyTheFish ManyTheFish

Labels
bug Something isn't working as expected error handler Issues related to the returned errors in Meilisearch v1.0.0 PRs/issues solved in v1.0.0 released on 2023-02-06
Projects
None yet
Milestone
v1.0.0
Development

Successfully merging a pull request may close this issue.

Fix matching strategy error meilisearch/meilisearch
3 participants
@ManyTheFish @curquiza @meili-bot