Chat say functionality #41

Merged
modulatingforce merged 14 commits from chat-say into main 2024-03-24 00:00:48 -04:00

I'm sure I'll want that instead of reply for routines functionality, so might as well go through a smol pr for this

Following documentation may be related

https://docs.rs/twitch-irc/5.0.1/twitch_irc/client/struct.TwitchIRCClient.html#method.say

Related area to update

Lines 104 to 108 in 45e3f02
async fn _say(&self, _: String, _: String) {
// more info https://docs.rs/twitch-irc/latest/twitch_irc/client/struct.TwitchIRCClient.html#method.say
// self.client.say(msg,outmsg).await.unwrap();
}


Plan of Action:

  • review current implementation of say_in_reply
  • review implementation of build in say
  • design & code say
  • unit tests? - Skipping - no need for this change
I'm sure I'll want that instead of `reply` for routines functionality, so might as well go through a smol pr for this Following documentation may be related https://docs.rs/twitch-irc/5.0.1/twitch_irc/client/struct.TwitchIRCClient.html#method.say Related area to update https://git.flake.sh/modulatingforce/forcebot_rs/src/commit/45e3f022973acf98310e7189d8d30dd81905a291/src/core/chat.rs#L104-L108 --- # Plan of Action: - [x] review current implementation of say_in_reply - [x] review implementation of build in say - [x] design & code say - [x] ~~unit tests?~~ - Skipping - no need for this change
modulatingforce self-assigned this 2024-03-23 01:57:18 -04:00
modulatingforce added this to the Rust Learning project 2024-03-23 01:57:22 -04:00
modulatingforce added the
Kind/Enhancement
Priority
Medium
labels 2024-03-23 01:59:28 -04:00
Author
Owner
  • #TODO Either implement below on current PR or create different PR

Thought of two interesting ideas , but unsure if I should put in here or have it's own pull request , after or during this say functionality

  1. a Button Command that would allow you to send a message in one chat, but also publish/direct it to another chat

  2. a Button Command that, when used in reply to an existing chat, publishes that message to another chat

- [x] _#TODO_ **Either implement below on current PR or create different PR** Thought of two interesting ideas , but unsure if I should put in here or have it's own pull request , after or during this say functionality 1. a Button Command that would allow you to send a message in one chat, but also publish/direct it to another chat 2. a Button Command that, when used in reply to an existing chat, publishes that message to another chat
modulatingforce added 3 commits 2024-03-23 13:33:04 -04:00
experimental say functionality
All checks were successful
ci/woodpecker/pr/cargo-checks Pipeline was successful
5802e9b755
modulatingforce added 8 commits 2024-03-23 20:10:52 -04:00
modulatingforce added 2 commits 2024-03-23 20:29:24 -04:00
clippy cleanup
All checks were successful
ci/woodpecker/pr/cargo-checks Pipeline was successful
060cfe6ba9
modulatingforce force-pushed chat-say from 060cfe6ba9 to e4a44894d7 2024-03-23 20:33:19 -04:00 Compare
modulatingforce added the
Status
Need More Info
label 2024-03-23 20:35:50 -04:00
Author
Owner

Looking good so far. Going to just think about this a bit if anything else should be added before I remove WIP flag to merge

Looking good so far. Going to just think about this a bit if anything else should be added before I remove WIP flag to merge
modulatingforce added 1 commit 2024-03-23 22:40:13 -04:00
added ts to experiment
All checks were successful
ci/woodpecker/pr/cargo-checks Pipeline was successful
97d76ea088
modulatingforce changed title from WIP: Chat say functionality to Chat say functionality 2024-03-23 23:52:53 -04:00
modulatingforce removed the
Status
Need More Info
label 2024-03-23 23:56:21 -04:00
modulatingforce merged commit c0e6bcc566 into main 2024-03-24 00:00:48 -04:00
modulatingforce deleted branch chat-say 2024-03-24 00:00:48 -04:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: modulatingforce/forcebot_rs#41
No description provided.