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

feat: add search endpoint #12

Draft
wants to merge 21 commits into
base: master
Choose a base branch
from
Draft

feat: add search endpoint #12

wants to merge 21 commits into from

Conversation

rkettelerij
Copy link
Collaborator

@rkettelerij rkettelerij commented Nov 29, 2024 •
edited
Loading

Description

Implement search/suggest endpoint

Type of change

  • New feature

Checklist:

  • I've double-checked the code in this PR myself
  • I've left the code better than before (boy scout rule)
  • The code is readable, comments are added that explain hard or non-obvious parts.
  • I've expanded/improved the (unit) tests, when applicable
  • I've run (unit) tests that prove my solution works
  • There's no sensitive information like credentials in my PR

@rkettelerij rkettelerij force-pushed the search-suggest-endpoint branch from 0be3d2d to 927cd70 Compare December 4, 2024 12:04
@rkettelerij rkettelerij changed the title feat: add search, implement autosuggest endpoint feat: add search endpoint Dec 4, 2024
@rkettelerij
feat: add search endpoint - make args with defaults optional
5598b60
@rkettelerij
feat: add search endpoint - return more columns in select in orde to …
1ec613b 
…build up GeoJSON response
@rkettelerij
feat: add search endpoint - weird test failure, fixing
6d3da80
@rkettelerij
feat: add search endpoint - return GeoJSON response
100655f
@rkettelerij
Merge branch 'master' into search-suggest-endpoint
d5a83cc 
# Conflicts:
#	internal/etl/etl.go
@rkettelerij
feat: add search endpoint - sync with master
a06da94
@rkettelerij
feat: add search endpoint - linting
09fd6ac
@rkettelerij
feat: add search endpoint - add numberReturned
50ec5ee
@rkettelerij
feat: add search endpoint - add href to allow users to get the actual…
b0c3ab4 
… features from the OGC API
@rkettelerij
feat: add search endpoint - add self link (alternate links to HTML/JS…
927f96e 
…ON-FG can be added in the future). ALso changed endpoint to just 'search' in line with OGC API Features part 9 Test Search (early draft)
@rkettelerij
feat: add search endpoint - highlight display name, not suggest
4faae30
@rkettelerij
Merge branch 'master' into search-suggest-endpoint
2e36235 
# Conflicts:
#	cmd/main.go
@rkettelerij
feat: add search endpoint - merge
5c923f4
@rkettelerij
feat: add search endpoint - linting
f5805c9
@rkettelerij
feat: add search endpoint - refactor magic strings to constants + spl…
6806f81 
…it method
@rkettelerij
feat: add search endpoint - use prepared statement for query term
5b4fefc
@rkettelerij
feat: add search endpoint - search in specific collection
06879d2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@rkettelerij