Skip to content

Banchobot stat handling#88

Open
omkelderman wants to merge 4 commits into
Tillerino:masterfrom
omkelderman:banchobot-stat-handling
Open

Banchobot stat handling#88
omkelderman wants to merge 4 commits into
Tillerino:masterfrom
omkelderman:banchobot-stat-handling

Conversation

@omkelderman

Copy link
Copy Markdown
Collaborator

resolve usernames by sending STAT-commands to BanchoBot and parsing its response which contains the username and userid

@omkelderman
omkelderman force-pushed the banchobot-stat-handling branch from b3ac882 to ebfe664 Compare August 27, 2016 11:54
@Tillerino

Copy link
Copy Markdown
Owner

This needs a unit test. I wanna see you mock bby.

@omkelderman

Copy link
Copy Markdown
Collaborator Author

I will, at some point, probably not today 😛

@omkelderman

Copy link
Copy Markdown
Collaborator Author

Soon™

@Tillerino

Copy link
Copy Markdown
Owner

What's the status here? Sleepy?

@omkelderman

Copy link
Copy Markdown
Collaborator Author

dont expect anything fancy soon ^^

@omkelderman
omkelderman deleted the banchobot-stat-handling branch June 19, 2017 07:56
@omkelderman

Copy link
Copy Markdown
Collaborator Author

oh fuk, did I just really? I kinda bulk removed all my stale branches on my fork cuz it was annoying.... Did I really accedently remove this branch? OOOOOOOOOOOOOOPS

@omkelderman
omkelderman restored the banchobot-stat-handling branch June 20, 2017 18:08
@omkelderman omkelderman reopened this Jun 20, 2017
@omkelderman

Copy link
Copy Markdown
Collaborator Author

OMG I LOVE YOU GITHUB ❤️
It auto picked up the branch again when I pushed it to my remote again 😄

@Tillerino

Copy link
Copy Markdown
Owner

Damn, that's nice.

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.

2 participants