Skip to content

Fix geojson#3586

Merged
CaroFG merged 3 commits into
mainfrom
fix-geojson
Jun 2, 2026
Merged

Fix geojson#3586
CaroFG merged 3 commits into
mainfrom
fix-geojson

Conversation

@CaroFG

@CaroFG CaroFG commented May 21, 2026

Copy link
Copy Markdown
Contributor

Description

Fixes #3585

Checklist

For internal Meilisearch team member only:

For external maintainers

  • Did you use any AI tool while implementing this PR (code, tests, docs, etc.)? If yes, disclose it in the PR description and describe what it was used for. AI usage is allowed when it is disclosed.
  • Have you made sure that the title is accurate and descriptive of the changes?

Summary

Fixes documentation for GeoJSON filtering by correcting the filterable attribute from _geo to _geojson.

Changes

  • .code-samples.meilisearch.yaml: Added code sample geosearch_guide_filter_settings_2 demonstrating how to configure the neighborhoods index's filterable-attributes to include ["_geojson"] via a PUT request.

  • capabilities/geo_search/how_to/use_geojson_format.mdx: Updated the "Filtering and sorting with GeoJSON documents" section to:

    • Import the new code-sample component
    • Replace the inline bash example with the generated component
    • Correct the documentation to reference _geojson (instead of _geo) as the filterable attribute for GeoJSON documents
    • Maintain query operators (_geoRadius, _geoBoundingBox, _geoPolygon)

Fixes

Resolves issue #3585 by correcting the documented filterable attribute for GeoJSON, addressing feedback from multiple customers about the documentation error.

Review Change Stack

@CaroFG CaroFG requested review from ManyTheFish and j-meili May 21, 2026 13:25
@CaroFG CaroFG requested a review from a team as a code owner May 21, 2026 13:25
@mintlify

mintlify Bot commented May 21, 2026

Copy link
Copy Markdown

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
meilisearch-documentation 🟢 Ready View Preview May 21, 2026, 1:27 PM

💡 Tip: Enable Workflows to automatically generate PRs for you.

@coderabbitai

coderabbitai Bot commented May 21, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: a9d01fb0-a038-42f7-bdbf-82fc7dd479d6

📥 Commits

Reviewing files that changed from the base of the PR and between 89316b6 and b06f356.

📒 Files selected for processing (2)
  • .code-samples.meilisearch.yaml
  • capabilities/geo_search/how_to/use_geojson_format.mdx

📝 Walkthrough

Walkthrough

This PR corrects documentation for GeoJSON filtering in Meilisearch by updating the referenced filterable attribute from _geo to _geojson. A new code sample is added to .code-samples.meilisearch.yaml demonstrating the correct curl command, and the MDX guide is updated to import and display that component alongside corrected attribute guidance.

Changes

GeoJSON Documentation Correction

Layer / File(s) Summary
GeoJSON filterable attribute correction with code sample
.code-samples.meilisearch.yaml, capabilities/geo_search/how_to/use_geojson_format.mdx
New YAML code sample entry demonstrates setting ["_geojson"] in filterable-attributes via curl. MDX documentation imports the generated component and updates text to reference _geojson instead of _geo in the filtering and sorting section.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A whisker-twitch of joy, I say,
The geo-docs are fixed today!
From _geo false to _geojson true,
Our customers now know just what to do!

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title is vague and generic, using 'Fix geojson' without specifying what the fix entails or which documentation is being corrected. Use a more descriptive title that clarifies the specific fix, such as 'Correct filterable attribute from _geo to _geojson in documentation'.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Linked Issues check ✅ Passed The PR successfully addresses issue #3585 by correcting the filterable attribute from _geo to _geojson in the documentation and accompanying code sample.
Out of Scope Changes check ✅ Passed All changes are directly scoped to fixing issue #3585: updating the filterable attribute documentation and adding the corresponding code sample.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix-geojson

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

ESLint skipped: no ESLint configuration detected in root package.json. To enable, add eslint to devDependencies.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@ManyTheFish ManyTheFish left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

sorry for the delay

LGTM

@CaroFG CaroFG added this pull request to the merge queue Jun 2, 2026
Merged via the queue into main with commit e2119c5 Jun 2, 2026
3 of 4 checks passed
@CaroFG CaroFG deleted the fix-geojson branch June 2, 2026 14:59
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.

Wrong filterable attribute for GeoJson

2 participants