Skip to content
This repository has been archived by the owner on Aug 30, 2022. It is now read-only.

Exclude Context Messages for Rocket.Chat Search #278

Open
westei opened this issue Sep 4, 2018 · 4 comments
Open

Exclude Context Messages for Rocket.Chat Search #278

westei opened this issue Sep 4, 2018 · 4 comments
Assignees
@westei
Labels
enhancement
Milestone
v0.9.0

Comments

@westei
Copy link
Member

westei commented Sep 4, 2018

Expected Behaviour

The #276 Rocket.Chat Query Provider uses some messages as the context for extracting the most relevant terms to be used for getting relevant messages.

The Rocket.Chat search now tends to return those exact messages (as they obviously best match the context).

Because of that we need to provide a way to explicitly exclude Messages used for the context from Search results.
@westei westei self-assigned this Sep 4, 2018
@westei westei added this to the v0.8.1 milestone Sep 12, 2018
@westei
Copy link
Member Author

westei commented Sep 12, 2018

Marked this as v0.8.1 as this will require changes in Chatpal and any changes in Chatpal depend on acceptance of a Pull requests by Rocket.Chat and a Rocket.Chat release.

westei added a commit that referenced this issue Sep 12, 2018
@westei
Smarti Backend implementation of #278: The Rockt.Chat Search Query in…
2cba0dd 
…cludes now the IDs of the messages used for the context. So that they can be excluded from the results
@westei westei modified the milestones: v0.8.1, v0.8.0 Sep 20, 2018
@westei
Copy link
Member Author

westei commented Sep 24, 2018

see chatpal-search-solr-ext #5 for details about the chatpal part of this issue

@westei westei modified the milestones: v0.8.0, v0.8.1 Sep 27, 2018
@Peym4n Peym4n mentioned this issue Apr 4, 2019
Closed
@Peym4n
Copy link
Contributor

Peym4n commented Apr 9, 2019

@westei To exclude the current room I would need the room ID in the smarti response.

Peym4n pushed a commit that referenced this issue Apr 9, 2019
#278: Implement context messages exclusion for rocketchat search (cha…
2dedf44 
…tpal)
westei added a commit that referenced this issue Apr 10, 2019
@westei
added the channel_id to rocketChat search QeruyBuilder responses #278
54fb023
Peym4n pushed a commit that referenced this issue Apr 10, 2019
#278: Implement current room exclusion for rocketchat search (chatpal)
4a5742a
@westei westei modified the milestones: v0.8.1, v0.9.0 Apr 11, 2019
@westei
Copy link
Member Author

westei commented Apr 11, 2019

Notes:

@westei westei mentioned this issue Apr 11, 2019
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@westei westei

Labels
enhancement
Projects
None yet
Milestone
v0.9.0
Development

No branches or pull requests
2 participants
@westei @Peym4n