Skip to content

Added Route guildMembersSearchV2 for POST /guilds/{guild.id}/members-search #1307

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.

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Lixqa
Copy link

@Lixqa Lixqa commented Jul 22, 2025

Please describe the changes this PR makes and why it should be merged:
This PR adds support for the POST /guilds/{guild.id}/members-search route, which enables searching for guild members using a variety of query parameters - not just by nickname as with the older endpoint.

This is a preparatory step toward reworking GuildMemberManager#search() in discord.js to utilize this newer, more flexible route. By doing so, we can offer developers greater control and precision when searching for guild members.

The name of the route includes "V2", even if it's not ideal. This just follows the route naming 🤷‍♂️.

If applicable, please reference Discord API Docs PRs or commits that influenced this PR:
/

@Lixqa Lixqa requested review from a team as code owners July 22, 2025 11:56
Copy link

vercel bot commented Jul 22, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
discord-api-types ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 24, 2025 0:11am

@vladfrangu
Copy link
Member

vladfrangu commented Jul 22, 2025

This endpoint isn't documented on Discord's end right? If not, I'm torn on documenting it, and if we do, it MUST be documented with @unstable.

@Lixqa
Copy link
Author

Lixqa commented Jul 22, 2025

No it isn't documented. It didn't even worked for bots like 3 months ago when I tested it last time. Seems like they enabled it for bots now. Maybe they are working on documenting it right now.

Sign up for free to 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