Skip to content
This repository has been archived by the owner on Mar 21, 2024. It is now read-only.

Add proximity precision setting #261

Merged
merged 7 commits into from
Jan 15, 2024
Merged

Conversation

ManyTheFish
Copy link
Member

@ManyTheFish ManyTheFish commented Dec 19, 2023

Summary

Define the proximityPrecision setting.


Changes

  • Add a new file with the setting details
  • modify the global index setting spec
  • modify the telemetry spec

Copy link
Contributor

@macraig macraig left a comment

Choose a reason for hiding this comment

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

The spec is super clear and easy to read, thanks @ManyTheFish ! ✨

meili-bors bot added a commit to meilisearch/meilisearch that referenced this pull request Jan 8, 2024
4303: Display default value when proximityPrecision is not set r=dureuill a=ManyTheFish

# Pull Request

## Related
Issue: #4187
Spec change requests: meilisearch/specifications#261 (comment)

## What does this PR do?
- Display default value when proximityPrecision is not set instead of Null


Co-authored-by: ManyTheFish <many@meilisearch.com>
ManyTheFish and others added 3 commits January 8, 2024 17:32
Co-authored-by: Maria Craig <maria@meilisearch.com>
Co-authored-by: Maria Craig <maria@meilisearch.com>
Co-authored-by: Maria Craig <maria@meilisearch.com>
text/0123-settings-api.md Outdated Show resolved Hide resolved
text/0123-proximity-precision.md Outdated Show resolved Hide resolved
text/0123-proximity-precision.md Outdated Show resolved Hide resolved
@macraig macraig merged commit b099d1b into release-v1.6.0 Jan 15, 2024
2 checks passed
@macraig macraig deleted the proximity-precision branch January 15, 2024 19:24
macraig added a commit that referenced this pull request Jan 15, 2024
* Bump API version

* Add proximity precision setting (#261)

* Add proximity precision setting

* Update text/0034-telemetry-policies.md

Co-authored-by: Maria Craig <maria@meilisearch.com>

* Update text/0034-telemetry-policies.md

Co-authored-by: Maria Craig <maria@meilisearch.com>

* Update text/0034-telemetry-policies.md

Co-authored-by: Maria Craig <maria@meilisearch.com>

* Update text/0123-settings-api.md

* Update text/0123-proximity-precision.md

* Update text/0123-proximity-precision.md

---------

Co-authored-by: Maria Craig <maria@meilisearch.com>

* write the task queue webhook specification (#262)

---------

Co-authored-by: María <maria@Marias-MacBook-Pro.local>
Co-authored-by: Many the fish <legendre.maxime.isn@gmail.com>
Co-authored-by: Tamo <tamo@meilisearch.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Ready For Review Feature specification must be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants