-
Notifications
You must be signed in to change notification settings - Fork 69
Allow to search users by member_uid #354
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
Conversation
@@ -83,7 +83,7 @@ | |||
] | |||
end | |||
|
|||
config.include Devise::TestHelpers, type: :controller | |||
config.include Devise::Test::ControllerHelpers, type: :controller |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Current build is "unstable" due to others test cases (not related to these changes), as noted by @rewritten in #351: cd25968. |
Take a look at @rewritten 's PR cause it looks like he's working on it. |
👍 seems that it was solved there, I'll rebase on top when it get merged to develop, because both branches are also touching the ransack form. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you please rebase from develop
@markets ?
Thanks!!!! 😍
Conflicts: app/views/users/index.html.erb
Updated from |
…ack/wiki/Using-Ransackers#4-convert-an-integer-database-field-to-a-string-in-order-to-be-able-to-use-a-cont-predicate-instead-of-the-usual-eq-which-works-out-of-the-box-with-integers-to-find-all-records-where-an-integer-field-id-in-this-example-contains-an-input-string) - Added sortable column on member_uid - Removed a couple of old comments
a17471f
to
22cda5d
Compare
@enricostano @sauloperez Finally CI is green ✅ 😃 Ready for review 👀 |
Conflicts: app/views/users/index.html.erb
Updated with latest |
I changed the label to "need review" because you didn't merge it and I can not test it |
Deployed on https://staging.timeoverflow.org/ |
Tested with @sseerrggii 🍏 |
Closes #343
Implemented using current search based on
Ransack
. I followed this guide to map the integer to a char and make it work with the pg ILIKE.EXTRA Allow to sort by the
member_uid
column