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

Fixes converting list of strings to ID #422

Merged
merged 3 commits into from
Sep 18, 2024
Merged

Fixes converting list of strings to ID #422

merged 3 commits into from
Sep 18, 2024

Conversation

joewesch
Copy link
Contributor

Closes: #421

If we are converting multiple items to ID and the input is a list of strings, instead of sending the query one item at a time (list_item) we were sending the full list of items each time (search):

else:
# Reminder: this get checks the QUERY_TYPES constant above, if the item is not in the list
# of approved query types, then it defaults to a q search
temp_dict = {QUERY_TYPES.get(k, "q"): search}

@joewesch
Copy link
Contributor Author

I have also bundled in the necessary change log information so we can quickly release this patch after merge.

@joewesch
Copy link
Contributor Author

@joewesch joewesch merged commit e552624 into develop Sep 18, 2024
27 checks passed
@joewesch joewesch deleted the u/joewesch_421 branch September 18, 2024 13:07
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.

admin_permission doesn't allow to set multiple groups
2 participants