~delthas/senpai-dev

This thread contains a patchset. You're looking at the original emails, but you may wish to use the patch review UI. Review patch
1

[PATCH] fix /msg command only sending first word

Details
Message ID
<20220719215500.271-1-gildarts@orbital.rocks>
DKIM signature
missing
Download raw message
Patch: +2 -2
---
 commands.go | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/commands.go b/commands.go
index 072c433..b8db69f 100644
--- a/commands.go
+++ b/commands.go
@@ -341,8 +341,8 @@ func commandDoMe(app *App, args []string) (err error) {

func commandDoMsg(app *App, args []string) (err error) {
	target := args[0]
	content := args[1]
	return commandSendMessage(app, target, content)
	content := args[1:]
	return commandSendMessage(app, target, strings.Join(content, " "))
}

func commandDoNames(app *App, args []string) (err error) {
-- 
2.36.1.windows.1
Details
Message ID
<23ea9e90fec9dc69647e24b291d9d264@dille.cc>
In-Reply-To
<20220719215500.271-1-gildarts@orbital.rocks> (view parent)
DKIM signature
missing
Download raw message
Thanks!

It's actually more of a bug in fieldsN which should properly return an 
array of size MaxArgs without splitting the last field.
I've pushed a fix that takes care of a bug, but by fixing fieldsN rather 
than the command.

To git.sr.ht:~taiite/senpai
    e740a6f..b3f64ae  master -> master
Reply to thread Export thread (mbox)