devzat icon indicating copy to clipboard operation
devzat copied to clipboard

Make messages a new struct / in general make stuff less hacky

Open quackduck opened this issue 4 years ago • 6 comments

Then we could have better detection if messages are dms, instead of the hacky way (as I note in the comment) we do this detection rn:

if strings.HasSuffix(senderName, " <- ") || strings.HasSuffix(senderName, " -> ") { // kinda hacky

https://github.com/quackduck/devzat/blob/main/devchat.go#L265

quackduck avatar Jun 14 '21 15:06 quackduck

We could also have some other fun stuff with this feature, plus cleaner and more oop I guess

quackduck avatar Jun 14 '21 15:06 quackduck

Yeah for example you could break out the dm to an own command like ./dm @someone <message>?

patrickhener avatar Jun 16 '21 11:06 patrickhener

Actually we had that exact command but moved to = cause it’s easier to type

quackduck avatar Jun 16 '21 11:06 quackduck

@patrickhener so we already have that lol

quackduck avatar Jun 22 '21 08:06 quackduck

fine

patrickhener avatar Jun 22 '21 12:06 patrickhener

yup

quackduck avatar Aug 01 '23 01:08 quackduck