Skip to content
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

Guild routes #593

Draft
wants to merge 12 commits into
base: dev
Choose a base branch
from
Draft

Guild routes #593

wants to merge 12 commits into from

Conversation

kozabrada123
Copy link
Member

@kozabrada123 kozabrada123 commented Jan 5, 2025

Working on #43

Additions

  • Guild::modify_mfa_level
  • Guild::search_members - this API is not great
  • Guild::get_members_supplemental

Changes

  • Guild::get - add with_counts
  • Guild::create - update schema for new fields, see POST /guilds/ schema is outdated spacebarchat/server#1251
  • Guild::modify - update schema, add audit_log_reason
  • Guild::get_members - update query parameters, apply them properly, documentation updates
  • rename Guild::search_members to Guild::query_members, since search members is another different route
  • Guild::query_members - update query parameters, apply them properly, documentation updates

Fixes

  • try to fix errors in our doctests, so they can actually be tested
  • fix an issue with .well-known discovery of instances not working
  • Guild::get_preview - method is GET, not PATCH

- updates Guild::get, Guild::create, Guild::modify, Guild::get_preview

- adds Guild::modify_mfa_level
@kozabrada123 kozabrada123 added this to the 0.19.0 milestone Jan 5, 2025
- renames Guild::search_members to Guild::query_members, since that's another, different route

- updates Guild::get_members and Guild::query_members
src/types/schema/guild.rs Fixed Show fixed Hide fixed
src/types/schema/guild.rs Fixed Show fixed Hide fixed
src/types/schema/guild.rs Fixed Show fixed Hide fixed
This api is pretty bad to translate to a strong typed language, if I get a better idea I may revisit it

Generics don't work, since you need the annotation for chrono::DateTime
src/types/entities/guild_member.rs Fixed Show fixed Hide fixed
src/types/entities/guild_member.rs Fixed Show fixed Hide fixed
src/types/entities/guild_member.rs Fixed Show fixed Hide fixed
src/types/entities/guild_member.rs Fixed Show fixed Hide fixed
src/types/schema/guild.rs Fixed Show fixed Hide fixed
src/types/schema/guild.rs Fixed Show fixed Hide fixed
src/types/schema/guild.rs Fixed Show fixed Hide fixed
src/types/schema/guild.rs Fixed Show fixed Hide fixed
src/api/guilds/guilds.rs Fixed Show fixed Hide fixed
src/api/guilds/guilds.rs Fixed Show fixed Hide fixed
src/api/guilds/guilds.rs Fixed Show fixed Hide fixed
src/api/guilds/guilds.rs Fixed Show fixed Hide fixed
src/api/guilds/guilds.rs Fixed Show fixed Hide fixed
src/types/entities/guild_member.rs Fixed Show fixed Hide fixed
src/types/entities/guild_member.rs Fixed Show fixed Hide fixed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant