Skip to content

Use the Search API in the issue search example - #1166

Merged
acrobat merged 1 commit into
KnpLabs:masterfrom
jihtsan:codex/1118-search-issues-example
Oct 23, 2026
Merged

acrobat merged 1 commit into
KnpLabs:masterfrom
jihtsan:codex/1118-search-issues-example

Conversation

@jihtsan

@jihtsan jihtsan commented Oct 23, 2026

Copy link
Copy Markdown
Contributor

The issue-search example calls the removed Issue::find() method and fails when copied into current clients. Use the supported Search API, filter closed issues in the specified repository, and extract the current page from the response items field.

Closes #1118.

Validation: extracted and syntax-checked the PHP snippet, then executed it against the repository’s actual Search class with a checked local HTTP dispatch stub; API selection, /search/issues query and item extraction passed. The equivalent read-only GitHub API query returned the documented response envelope and closed issues, without pull requests. git diff --check and independent Standards/Spec reviews passed. The complete library test suite and other PHP versions were not run for this documentation-only change.

Closes KnpLabs#1118. Replace the removed Issue::find call and extract issues from the search response items, with an is:issue filter and first-page semantics.
@acrobat
acrobat merged commit 3f22084 into KnpLabs:master Oct 23, 2026
10 of 11 checks passed
@acrobat

acrobat commented Oct 23, 2026

Copy link
Copy Markdown
Collaborator

Thanks @jihtsan! And congrats on your first contribution! 🎉

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.

api('issue')->find() is still documented but was removed

2 participants