Skip to content

Conversation

rvmourik
Copy link
Collaborator

@rvmourik rvmourik commented Mar 1, 2023

Also simplified the code to not using nested values, this is not going to happen in a global search pattern

Also simplified the code to not using nested values, this is not going to happen in a global search pattern
@rvmourik rvmourik requested a review from darylholling March 1, 2023 19:56
@rvmourik
Copy link
Collaborator Author

rvmourik commented Mar 1, 2023

@darylholling dit lost nog niet de nested problemen met associations op, maar wel het probleem dat een combinatie van een reguliere filter i.c.m. een global search niet werkt. Voor deze fix deed hij dan dit:

reguliere_filter = 1 OR global_search_prop1 = 2 OR global_search_prop3 = 2

Waardoor er nooit resultaten uitgefilterd werden, na deze fix is het weer:

reguliere_filter = 1 AND (global_search_prop1 = 2 OR global_search_prop3 = 2)


Als hij functioneert, graag approven/mergen en een release taggen.

@darylholling darylholling merged commit 305d2fe into main Mar 7, 2023
@darylholling darylholling deleted the fix/global-search-and-or branch March 7, 2023 09:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants