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

[new] global search across channel #7274

Closed
wants to merge 2 commits into from
Closed

[new] global search across channel #7274

wants to merge 2 commits into from

Conversation

cyberhck
Copy link
Contributor

@RocketChat/core

Closes #1615

it's really difficult to explain with screenshots, basically this PR enables global search without breaking existing functionality. One optional parameter is now passed to enable global search.

This is my first time doing meteor, so I will need a review.

If there are any changes you'd like, please let me know and I'll fix it.

Thanks.

@CLAassistant
Copy link

CLAassistant commented Jun 17, 2017

CLA assistant check
All committers have signed the CLA.

@cyberhck
Copy link
Contributor Author

Anything going on with this PR?

@gdelavald gdelavald self-assigned this Jun 19, 2017
RocketChat.MessageAction.hideDropDown();
return RoomHistoryManager.getSurroundingMessages(message, 50);
window.setTimeout(() => {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure if this is necessary.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ahh, if the connection isn't very good, it sometimes takes some time to go to other room, and it was giving errors, is there a promise or callback which I can use?

I tried without setTimeout because this feels very dirty, it worked the first time because it was instant, then it stopped working.

@gdelavald
Copy link
Contributor

Hey @cyberhck, thanks for the Pull Request!
Looks like this change makes the default search to global, I think it would be very cool to be able to toggle that in the search interface (with a checkbox perhaps?).

@cyberhck
Copy link
Contributor Author

@gdelavald I was definitely thinking that.

I looked into slack and other chats, there were only global ones, and since I'm poor in designing, I wasn't so sure whee to put the checkbox in :)

If you have any suggestions, I'd be more than happy to update.
Thanks :)

@gdelavald
Copy link
Contributor

I think the easier/simpler way would be doing something like this:
captura de tela 2017-06-20 as 11 50 09
We can discuss style later, I would like to see the functionality implemented first.
Also, the search result ends up being a big list without displaying from which channel came which message, I think that would be pretty interesting to see as well.

@cyberhck
Copy link
Contributor Author

Okay, I'll keep add this to my do list for tonight :) thanks for feedback

@cyberhck
Copy link
Contributor Author

I've been quite busy lately, I'll give this a go again when I have a little breathing room, or if anyone else wants to take it from here, I'm okay with that, (don't care for the bounty, whoever finishes may take it 🙂 )

@Cu57arD
Copy link

Cu57arD commented Jul 31, 2017

I have added $200 to the bounty - https://www.bountysource.com/issues/28998888-global-search-across-channels

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.

Global search across channels
5 participants