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 First Message support to Twitch user messages. #168

Merged
merged 4 commits into from
Nov 9, 2021

Conversation

Cidan
Copy link
Contributor

@Cidan Cidan commented Nov 8, 2021

Twitch now sends to moderators and stream owners a new tag with all chat messages, "first-msg=N" where N is 0 or 1. If it's 1, this is the first time this user chatted on the target user's stream. This CL adds support for parsing that tag, and adds a new boolean to the private message struct called FirstMessage to reflect this status.

Twitch now sends to moderators and stream owners a new tag with all chat messages, "first-msg=N" where N is 0 or 1. If it's 1, this is the first time this user chatted on the target user's stream. This CL adds support for parsing that tag, and adds a new boolean to the private message struct called FirstMessage to reflect this status.
@coveralls
Copy link

coveralls commented Nov 8, 2021

Coverage Status

Coverage increased (+0.009%) to 97.92% when pulling caf8c4a on Cidan:master into b173bfc on gempir:master.

message_test.go Outdated Show resolved Hide resolved
message.go Outdated Show resolved Hide resolved
@Cidan
Copy link
Contributor Author

Cidan commented Nov 9, 2021

Updated tests to be TDD and cleaned up the first message check. PTAL.

@gempir gempir merged commit 61d050b into gempir:master Nov 9, 2021
@gempir
Copy link
Owner

gempir commented Nov 9, 2021

Thank you for the contribution! I dig the table test

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

3 participants