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

Implement admin api #1106

Closed
wants to merge 14 commits into from
Closed

Implement admin api #1106

wants to merge 14 commits into from

Conversation

Puyodead1
Copy link
Contributor

@Puyodead1 Puyodead1 commented Dec 17, 2023

Added Routes:

  • /admin/guilds/:guild_id/bans
    • GET - get guild bans
  • /admin/guilds/:guild_id/bans/:user_id
    • GET - get a guild ban for a user
    • PUT - ban a user from the guild
    • DELETE - unban a user from the guild
  • /admin/guilds/:guild_id
    • GET - get a guild
    • PATCH - update a guild
    • DELETE - delete a guild
  • /admin/guilds/:guild_id/members
    • GET - get guild members
    • PUT - add a user to a guild
  • /admin/guilds/:guild_id/members/:user_id
    • PATCH - update a guild members
    • DELETE - remove a member from a guild
  • /admin/guilds
    • GET (paginated) - get instance guilds
    • POST - create a guild

    This is different from the normal guild create endpoint, this one allows creating a guild for another user

  • /admin/users/:user_id
    • GET - get a user
    • PATCH - update a user
    • DELETE - delete a user
  • /admin/users
    • GET (paginated)

Modified Routes:

  • /guilds/:guild_id/members/:user_id
    • GET - get a guild member, bypasses the requirement of being a member

New Rights:

  • ADMIN_CREATE_USERS
  • ADMIN_READ_USERS
  • ADMIN_UPDATE_USERS
  • ADMIN_DELETE_USERS
  • ADMIN_CREATE_GUILDS
  • ADMIN_READ_GUILDS
  • ADMIN_UPDATE_GUILDS
  • ADMIN_DELETE_GUILDS
  • ADMIN_CREATE_MEMBERS
  • ADMIN_READ_MEMBERS
  • ADMIN_UPDATE_MEMBERS
  • ADMIN_DELETE_MEMBERS
  • ADMIN_CREATE_GUILD_BANS
  • ADMIN_READ_GUILD_BANS
  • ADMIN_UPDATE_GUILD_BANS
  • ADMIN_DELETE_GUILD_BANS

@Puyodead1 Puyodead1 marked this pull request as ready for review December 18, 2023 00:03
@Puyodead1 Puyodead1 closed this Dec 18, 2023
@TheArcaneBrony
Copy link
Member

just closed?

@Puyodead1
Copy link
Contributor Author

just closed?

I'm rewriting it differently, I mentioned in discord that I was going to instead modify existing routes/add new routes without the /admin prefix. see: e348872

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.

2 participants