Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@qingyashu
Copy link
Contributor

New Features

  • add "search" operator to filter var, to support fulltext search

@qingyashu qingyashu requested review from m0nhawk and mfshao November 26, 2019 16:53



<a name="filter-search"></a>
Copy link
Contributor

@m0nhawk m0nhawk Nov 27, 2019

Choose a reason for hiding this comment

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

Minor comment: you don't need to put the anchors in Markdown manually, all heading on Github already have them. See more here. So for "Basic filter unit" it would be: https://github.com/uc-cdis/guppy/blob/feat/search/doc/queries.md#basic-filter-unit

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ah good to know! thanks!

Copy link
Contributor

@m0nhawk m0nhawk left a comment

Choose a reason for hiding this comment

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

Looks fine for me. 👍

@qingyashu qingyashu requested a review from em-ingram November 27, 2019 20:32
Copy link
Contributor

@em-ingram em-ingram left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Contributor

@mfshao mfshao left a comment

Choose a reason for hiding this comment

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

cool! 👍

@qingyashu qingyashu merged commit 29a968f into master Dec 2, 2019
@m0nhawk m0nhawk deleted the feat/search branch December 3, 2019 05:44
@m0nhawk m0nhawk restored the feat/search branch December 3, 2019 05:44
@mfshao mfshao deleted the feat/search branch May 5, 2020 18:04
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.

5 participants