search

iSlammedMyKindle / twitchToDiscordBot

Cross-platform messaging interface between discord and twitch chats
0 stars 2 forks source link

(WIP): Replies #8

Closed AceLikesGhosts closed 1 year ago

AceLikesGhosts commented 1 year ago

Work in progress, no real progress has been made yet; draft. (:

AceLikesGhosts commented 1 year ago

Why did my linting in VSCode on my laptop format change it so that there is a space before parens (if (thing) {})

AceLikesGhosts commented 1 year ago

So, current options to make this work properly are :

  1. Monkeypatch the TMI library
  2. Swap to a different library (tuwuple prolly)
  3. Use the same solution we're using to delete messages, which is stupid.
AceLikesGhosts commented 1 year ago

Putting this PR on hold until we find a better way to get the proper message ID; Maybe time to swap from TMI to Puwuple? Time to make another draft PR for that Twitch wrapper migration?

iSlammedMyKindle commented 1 year ago

Oof... I hate that we can't use this. I'm very interested in reading about twuple with it being a new library in all

That's definitely one me - I guess for some background when reading about the twitch api, they had tmi.js as a direct reference, so I used that because that was what was in the docs.

Lesson learned perhaps - make sure people are actively working on the thing XP

This is a cool prototype, nice work :+1: - I'll be stepping on your toes though tomorrow in order to implement the tying in of multiple twitch messages

AceLikesGhosts commented 1 year ago

Going to reopen this once the migration to Pwurple is done, due to not wanting to add jank code for IDs. (:

AceLikesGhosts commented 1 year ago

New PR for this incoming.