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
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
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 changed title from WIP: Chat say functionality to Chat say functionality 2024-03-23 23:52:53 -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.