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.

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

[BUG] SearchParameter Role.name must be token not string #515

Open
ruscoder opened this issue Jun 2, 2022 · 0 comments
Open

[BUG] SearchParameter Role.name must be token not string #515

ruscoder opened this issue Jun 2, 2022 · 0 comments
Assignees
Labels

Comments

@ruscoder
Copy link

ruscoder commented Jun 2, 2022

We have some queries something like get /Patient?_has:Role:patient:name=patient and it works pretty slow when Role.name is string search param type. Currently solved the issue by manually creating new name-token search param with proper type, but it's very inconvinient. I believe that such small change is really easy to do in the core.

get /Role?name=patient

  - >-
    SELECT "role".* FROM "role" WHERE
    aidbox_text_search(knife_extract_text("role".resource,
    $JSON$[["name"]]$JSON$)) ilike unaccent(?) LIMIT ? OFFSET ? 
  - '% test%'
  - 100
  - 0
@ruscoder ruscoder added the bug label Jun 2, 2022
@krevedkokun krevedkokun self-assigned this Aug 11, 2022
@krevedkokun krevedkokun added this to the August 2022 - v:2208 milestone Aug 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants