rhodecode-enterprise-ce Pull Request !2276

Closed

Pull request !2276 Created on by

Needs more input validation & tests, but works great for integration with internal bug tracker.

Commit flow:

Versions:

Pull request versions not available.

ver	Time	Author	Commit	Description
`latest`	Tue, 26 Mar 2019 14:58:58	pc	`r3508:469847094947`	Issue #5548 - Pass repo_group_name
`latest`	Tue, 26 Mar 2019 14:52:34	pc	`r3507:0208bd4c9de8`	Issue #5548 - Add repo_group_name and docs to search API
`v1`	Thu, 21 Mar 2019 11:52:57	pc	`r3506:733c92d7db87`	Issue #5548 - Add initial search API method
3 commits hidden, click expand to show them.

No files

General Comments 9

v1 |

|

Auto status change to "Under Review"

note

jenkins-tests

v1 |

|

Build Succeeded!

note

marcink

v1 |

|

P.s we don't see a signed CLA for the contribution. If you wish to merge this please check: https://rhodecode.com/sign-cla

cla-bot

v1 |

|

CLA FOUND and APPROVED

|

Pull request updated. Auto status change to "Under Review"

Changed commits:
  * 2 added
  * 0 removed

Changed files:
  * M rhodecode/api/views/search_api.py

note

jenkins-tests

|

Build Succeeded!

cla-bot

|

CLA FOUND and APPROVED

marcink

|

Status change > Approved

marcink

|

Thanks for contribution. We'll merge this manually.

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository access permissions settings
t s	Toggle sidebar on some pages

	This file was removed from diff during updates to this pull-request. There are still outdated/unresolved comments attached to it.
					note marcink Thu, 21 Mar 2019 14:11:38 outdated `v1` `\|` Comment #21761 Edit Delete `\|` New 4.16 API allow repo_name, or repo_group_name scope filtering imho worth to add this. Please see the search() implementation: https://code.rhodecode.com/rhodecode-enterprise-ce/files/c75dd264c5f748c55e73391632ccd00a07f03f99/rhodecode/lib/index/whoosh.py#L102 note marcink Thu, 21 Mar 2019 14:13:15 outdated `v1` `\|` Comment #21762 Edit Delete `\|` for usage would be great to list the 3 types available, otherwise people need to guess. `'content', 'path', 'commit'`