Skip to content

[11.9] Add support for Advanced Search API #696

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 3 commits into from
Closed

[11.9] Add support for Advanced Search API #696

wants to merge 3 commits into from

Conversation

IAL32
Copy link
Contributor

@IAL32 IAL32 commented Feb 16, 2022

The Advanced Search API can be called as follows:

(GET) /search

Attribute Type Required Description
scope string yes The scope to search in
search string yes The search query
state string no Filter by state. Issues and merge requests are supported; it is ignored for other scopes.
confidential boolean no Filter by confidentiality. Issues scope is supported; it is ignored for other scopes.
order_by string no Allowed values are created_at only. If this is not set, the results are either sorted by created_at in descending order for basic search, or by the most relevant documents when using advanced search.
sort string no Allowed values are asc or desc only. If this is not set, the results are either sorted by created_at in descending order for basic search, or by the most relevant documents when using advanced search.

This PR adds support for this API.

@IAL32 IAL32 changed the base branch from 11.7 to 11.8 February 16, 2022 11:56
GrahamCampbell
GrahamCampbell requested changes Apr 24, 2022
View reviewed changes
src/Api/Projects.php
@@ -78,6 +78,8 @@ public function all(array $parameters = [])
'id', 'name', 'path', 'created_at', 'updated_at', 'last_activity_at',
'repository_size', 'storage_size', 'packages_size', 'wiki_size',
];
$resolver->setDefined('pagination');
Copy link
Member

Choose a reason for hiding this comment

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

These parameters are not intentionally not supported. Please remove them. You need to use the result pager class.

@GrahamCampbell GrahamCampbell changed the base branch from 11.8 to 11.9 April 24, 2022 19:04
@GrahamCampbell GrahamCampbell changed the title [11.8] Add support for Advanced Search API [11.9] Add support for Advanced Search API Apr 24, 2022
@stale
Copy link

stale bot commented Aug 31, 2022

This issue has been automatically marked as stale because there has been no recent activity. It will be closed after 30 days if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Aug 31, 2022
@stale stale bot closed this Sep 30, 2022
@macbookandrew macbookandrew mentioned this pull request Nov 10, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@GrahamCampbell GrahamCampbell GrahamCampbell requested changes

Assignees
No one assigned
Labels
stale
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@IAL32 @GrahamCampbell