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
[CIS-929] UserSearch with given Query #1206
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1206 +/- ##
==========================================
- Coverage 91.25% 91.25% -0.01%
==========================================
Files 217 217
Lines 9277 9292 +15
==========================================
+ Hits 8466 8479 +13
- Misses 811 813 +2
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
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.
LGTM ✅ Great work 👍
print("### search with query :\(query)") | ||
userQueryUpdater.update(userListQuery: query, policy: .replace) { error in | ||
print("### search complete") |
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.
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.
Definitely 😓
So it's possible to make custom searches and usable with multi-tenancy
With this change, users can customize the suggestions search query, most apparent example is multi-tenancy (it doesn't work as of now)
aee3790
to
970ebc1
Compare
With this PR, customizing mentions to work with multi-tenancy is possible. This also opens possibility of using UserSearchController is a variety of custom manners