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 user commands for looking up invites #7

Open
oleggtro opened this issue Sep 8, 2022 · 10 comments
Open

Implement user commands for looking up invites #7

oleggtro opened this issue Sep 8, 2022 · 10 comments
Assignees
Labels
enhancement New feature or request

Comments

@oleggtro
Copy link
Contributor

oleggtro commented Sep 8, 2022

No description provided.

@oleggtro oleggtro added the enhancement New feature or request label Sep 8, 2022
@Erik1000
Copy link
Member

Erik1000 commented Sep 8, 2022

Open for PRs

@oleggtro oleggtro self-assigned this Sep 8, 2022
@michihupf
Copy link

Any more context on usage and behaviour?

@oleggtro
Copy link
Contributor Author

oleggtro commented Nov 5, 2023

nah

ne also ich weiß selbst nicht mehr genau was ich damit gemeint hab, aber ich denk mal, dass das mit invite tracking zu tun hatte. Also dass man nachvollziehen kann, welches Mitglied von wem eingeladen wurde (ähnlich wie aufm pr0).

Es gibt eigentlich kein Design vorgegeben. Wenn du dich dem annehmen willst hast du eigentlich ziemlich offene Hand iirc

@michihupf
Copy link

Sicher dass das überhaupt geht? Ich glaube man kann von Invites doch nur den Count aufrufen oder?
https://discord.com/developers/docs/resources/invite#get-invite

@Erik1000
Copy link
Member

Erik1000 commented Nov 5, 2023

Sicher dass das überhaupt geht? Ich glaube man kann von Invites doch nur den Count aufrufen oder?

https://discord.com/developers/docs/resources/invite#get-invite

Invite tracking ist implementiert und funktioniert so, dass es versucht den Invite count vor und nach einem Join zu messen und der invite, der sich verändert hat, ordnet es dann zu.
Mittlerweile kann man aber sehen, wer welchen invite nutzt (also native im client).
In dem Issue ging es nur darum, die erfassten Daten via command zugänglich zu machen.

@michihupf
Copy link

Ist dieser Issue dann überhaupt noch notwendig wenn es native schon geht?

@Erik1000
Copy link
Member

Erik1000 commented Nov 6, 2023

Ja weil man nicht sehen kann wen man selbst eingeladen hat

@michihupf
Copy link

Und was ist mit dem list command? Macht der das nicht schon?

/// List invites created by you or another member
#[command(slash_command, ephemeral, required_bot_permissions = "MANAGE_GUILD")]
pub async fn list(
ctx: Context<'_>,
#[description = "The member you want to view the invites of"] member: Option<Member>,
) -> Result<()> {
match member {
Some(member) => {
match ctx
.guild()
.unwrap()
.member_permissions(ctx.discord().http(), ctx.author().id)
.await?
.manage_guild()
{
_ if ctx.author().id == member.user.id => {
list_invites(ctx, member.user.id, true).await
}
true => list_invites(ctx, member.user.id, true).await,
false => Err(anyhow!(
"You don't have the permission to list invites of other members."
)
.into()),
}
}
None => list_invites(ctx, ctx.author().id, false).await,
}
}

@Erik1000
Copy link
Member

Erik1000 commented Nov 6, 2023

Und was ist mit dem list command? Macht der das nicht schon?

/// List invites created by you or another member
#[command(slash_command, ephemeral, required_bot_permissions = "MANAGE_GUILD")]
pub async fn list(
ctx: Context<'_>,
#[description = "The member you want to view the invites of"] member: Option<Member>,
) -> Result<()> {
match member {
Some(member) => {
match ctx
.guild()
.unwrap()
.member_permissions(ctx.discord().http(), ctx.author().id)
.await?
.manage_guild()
{
_ if ctx.author().id == member.user.id => {
list_invites(ctx, member.user.id, true).await
}
true => list_invites(ctx, member.user.id, true).await,
false => Err(anyhow!(
"You don't have the permission to list invites of other members."
)
.into()),
}
}
None => list_invites(ctx, ctx.author().id, false).await,
}
}

Zeigt nur aktive invites. Keine die früher mal von jemanden benutzt wurden

@oleggtro
Copy link
Contributor Author

dead chat

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants