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

Added ability to see join date of other users #48

Closed
wants to merge 3 commits into from

Conversation

fooooooooooooooo
Copy link

No description provided.

@DarkZek
Copy link
Contributor

DarkZek commented Nov 19, 2018

You should just be using
IUser user = null
as the arguements and change the code depending on if the user is null or not. 0/10 foooo

@TheTrueColonel
Copy link
Contributor

image


[Command("joindate"), Summary("Display join date of mentioned user. Syntax : !joindate @user")]
[Command("joindate"), Summary("Display your join date or join date of mentioned user. Syntax : !joindate @user")]
private async Task JoinDate(IUser user)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

private async Task JoinDate(IUser user = null)

You're bad

@Sirush
Copy link
Owner

Sirush commented Nov 19, 2018

It's intentional that this doesn't exist as stated many times before.

@Sirush Sirush closed this Nov 19, 2018
@fooooooooooooooo fooooooooooooooo deleted the patch-1 branch October 21, 2019 11:29
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.

None yet

4 participants