Skip to content

Unified search implementation - #2200

Merged
juliusknorr merged 13 commits into
masterfrom
enh/search
Sep 1, 2020
Merged

Unified search implementation#2200
juliusknorr merged 13 commits into
masterfrom
enh/search

Conversation

@juliusknorr

Copy link
Copy Markdown
Member

No description provided.

@juliusknorr
juliusknorr marked this pull request as ready for review August 20, 2020 16:24
@juliusknorrjuliusknorr added this to the ⭐ 1.1.0 milestone Aug 20, 2020
@jakobroehrl

This comment has been minimized.

@juliusknorr

Copy link
Copy Markdown
MemberAuthor

Pushed a fix for that.

@jakobroehrl

Copy link
Copy Markdown
Contributor

Pushed a fix for that.

Now the search is working for the card title, card description, board name
Card comments aren't searched

@juliusknorr

Copy link
Copy Markdown
MemberAuthor

Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
@juliusknorr

Copy link
Copy Markdown
MemberAuthor

Missing pieces for a follow up:

  • Pagination for search results
  • Search comments

@jakobroehrljakobroehrl left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This thing is working!

@juliusknorr
juliusknorr merged commit 3f30e20 into masterSep 1, 2020
@juliusknorr
juliusknorr deleted the enh/search branch September 1, 2020 11:07
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@juliusknorr@jakobroehrl