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

Update bool operator syntax for Elasticsearch simple_query_string and excludes #6360

Closed
1 task
patphongs opened this issue Jul 3, 2024 · 3 comments · Fixed by #6405 · May be fixed by #6363
Closed
1 task

Update bool operator syntax for Elasticsearch simple_query_string and excludes #6360

patphongs opened this issue Jul 3, 2024 · 3 comments · Fixed by #6405 · May be fixed by #6363
Assignees
@patphongs
Labels
Work: Front-end
Milestone
25.i

Comments

@patphongs
Copy link
Member

patphongs commented Jul 3, 2024 •
edited
Loading

Summary

What we're after:
We are considering updating our the way we parse and execute query strings by switching from query_string to simple_query_string in the API. With this change will require the CMS app to update how we are passing the API call through.

Syntax update

Current UI

((coordinated OR communications) OR (in-kind AND contributions) OR ("independent expenditure")) AND (-travel -authorization)

Web UI update

(coordinated | communications)|(in-kind + contributions)|("independent expenditure") -travel -authorization

API query transformation

q=((coordinated|communications)|(in-kind+%2bcontributions)|("independent+expenditure"))&q_exclude=(travel+authorization)

Related issues

List any relevant related issue(s)

Completion criteria

  • Query string operator syntax is transformed according to syntax update
@patphongs patphongs self-assigned this Jul 3, 2024
@patphongs patphongs modified the milestones: 25.4, 25.5 Jul 3, 2024
@patphongs patphongs mentioned this issue Jul 3, 2024
Closed
9 tasks
@patphongs patphongs mentioned this issue Jul 3, 2024
Draft
@patphongs patphongs modified the milestones: 25.5, 25.6 Jul 22, 2024
@patphongs
Copy link
Member Author

Moving since we are freezing JS work until webpack is in

@patphongs
Copy link
Member Author

Remove exclude boolean field from statutes search until we have documents indexed.

@patphongs patphongs mentioned this issue Jul 31, 2024
Merged
@pkfec pkfec modified the milestones: 25.6, 25.i Aug 6, 2024
@patphongs
Copy link
Member Author

Remove exclude boolean field from statutes search until we have documents indexed.

We are not going to address this at the moment since statutes may eventually get this functionality and it's too tightly coupled to our current global legal search to accommodate removing it.

@patphongs patphongs changed the title Update bool operator syntax for Elasticsearch simple_query_string Update bool operator syntax for Elasticsearch simple_query_string and excludes Aug 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@patphongs patphongs

Labels
Work: Front-end
Projects
Website project
Status: ✅ Done
Milestone
25.i
3 participants
@pkfec @patphongs @tmpayton