Skip to content

Feat/migrate to nominatim for search#926

Merged
scheidtdav merged 14 commits intodevfrom
feat/migrate-to-nominatim-for-search
May 7, 2026
Merged

Feat/migrate to nominatim for search#926
scheidtdav merged 14 commits intodevfrom
feat/migrate-to-nominatim-for-search

Conversation

@jona159
Copy link
Copy Markdown
Contributor

@jona159 jona159 commented May 6, 2026

Type of Change

  • Dependency upgrade
  • Bug fix (non-breaking change)
  • Breaking change
    • e.g. a fixed bug or new feature that may break something else
  • New feature
  • Code quality improvements
    • e.g. refactoring, documentation, tests, tooling, ...

Implementation

Checklist

  • I gave this pull request a meaningful title
  • My pull request is targeting the dev branch
  • I have added documentation to my code
  • I have deleted code that I have commented out

Additional Information

  • This PR closes #

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 6, 2026

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 69.22% 1775 / 2564
🔵 Statements 67.52% 1844 / 2731
🔵 Functions 69.24% 367 / 530
🔵 Branches 56.26% 948 / 1685
File CoverageNo changed files found.
Generated in workflow #2520 for commit eb1829a by the Vitest Coverage Report Action

Copy link
Copy Markdown
Member

@scheidtdav scheidtdav left a comment

Choose a reason for hiding this comment

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

Looks good, just some small notes.

Comment thread app/components/search/search-types.ts Outdated
Comment thread app/components/search/index.tsx Outdated
Comment thread app/components/search/index.tsx Outdated
Comment thread app/components/search/index.tsx
Comment thread app/components/search/index.tsx Outdated
@socket-security
Copy link
Copy Markdown

socket-security Bot commented May 7, 2026

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedisbot@​5.1.3910010010093100

View full report

@scheidtdav scheidtdav enabled auto-merge (squash) May 7, 2026 11:21
@scheidtdav scheidtdav merged commit d3ce253 into dev May 7, 2026
6 checks passed
@scheidtdav scheidtdav deleted the feat/migrate-to-nominatim-for-search branch May 7, 2026 11:26
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.

2 participants