GNU bug report logs - #54024
[mumi] Preprocess search query from GraphQL before use

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 54024 in the body.
You can then email your comments to 54024 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Arun Isaac <arunisaac <at> systemreboot.net>
To: Ricardo Wurmus <rekado <at> elephly.net>
Cc: Arun Isaac <arunisaac <at> systemreboot.net>, guix-patches <at> gnu.org
Subject: [mumi] Preprocess search query from GraphQL before use
Date: Wed, 16 Feb 2022 13:51:21 +0530

Hi Ricardo,

It looks like there is a small bug in the GraphQL patches I sent
earlier. I had failed to preprocess the search query before use.

Regards,
Arun

Arun Isaac (1):
  web: Preprocess search query from GraphQL before use.

 mumi/web/graphql.scm | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

-- 
2.34.0

Message #8 received at 54024 <at> debbugs.gnu.org (full text, mbox):

From: Arun Isaac <arunisaac <at> systemreboot.net>
To: Ricardo Wurmus <rekado <at> elephly.net>
Cc: 54024 <at> debbugs.gnu.org, Arun Isaac <arunisaac <at> systemreboot.net>
Subject: [PATCH] web: Preprocess search query from GraphQL before use.
Date: Wed, 16 Feb 2022 13:54:46 +0530

* mumi/web/graphql.scm (<query>): Preprocess search query using
process-query before passing on to search-bugs.
---
 mumi/web/graphql.scm | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/mumi/web/graphql.scm b/mumi/web/graphql.scm
index 662c5ba..56110a0 100644
--- a/mumi/web/graphql.scm
+++ b/mumi/web/graphql.scm
@@ -102,7 +102,8 @@
            (bug-status number)))
   (issues (non-nullable-type (list-type <issue>))
           (lambda* (parent context info #:key search)
-            (search-bugs search)))
+            (search-bugs (string-join
+                          (process-query search)))))
   (person (non-nullable-type <person>)
           (lambda* (parent context info #:key email)
             email)))
-- 
2.34.0

Message #13 received at 54024-done <at> debbugs.gnu.org (full text, mbox):

From: Ricardo Wurmus <rekado <at> elephly.net>
To: Arun Isaac <arunisaac <at> systemreboot.net>
Cc: 54024-done <at> debbugs.gnu.org
Subject: Re: [mumi] Preprocess search query from GraphQL before use
Date: Thu, 17 Feb 2022 23:19:28 +0100

Hi Arun,

> It looks like there is a small bug in the GraphQL patches I sent
> earlier. I had failed to preprocess the search query before use.

Applied, thanks!

(Reconfiguring issues.guix.gnu.org would take a little while longer as
we’re in the process of moving the root fs and /gnu/store to new disks.
Almost finished it today but ran into problems after rebooting.)

-- 
Ricardo

Message #16 received at 54024-done <at> debbugs.gnu.org (full text, mbox):

From: Arun Isaac <arunisaac <at> systemreboot.net>
To: Ricardo Wurmus <rekado <at> elephly.net>
Cc: 54024-done <at> debbugs.gnu.org
Subject: Re: [mumi] Preprocess search query from GraphQL before use
Date: Sat, 19 Feb 2022 12:01:46 +0530

[Message part 1 (text/plain, inline)]

Thanks, Ricardo! I think this new patch still isn't up on
issues.guix.gnu.org, but that's ok. It's not very urgent.

Should we consider more automated ways to redeploy mumi on every commit?
Or maybe, we should run a separate development server just for
that. WDYT?

[signature.asc (application/pgp-signature, inline)]

Message #19 received at 54024-done <at> debbugs.gnu.org (full text, mbox):

From: Ricardo Wurmus <rekado <at> elephly.net>
To: Arun Isaac <arunisaac <at> systemreboot.net>
Cc: 54024-done <at> debbugs.gnu.org
Subject: Re: [mumi] Preprocess search query from GraphQL before use
Date: Sat, 19 Feb 2022 12:07:20 +0100

Arun Isaac <arunisaac <at> systemreboot.net> writes:

> Thanks, Ricardo! I think this new patch still isn't up on
> issues.guix.gnu.org, but that's ok. It's not very urgent.

It’s not up because we’re still having problems with the migration.

> Should we consider more automated ways to redeploy mumi on every commit?
> Or maybe, we should run a separate development server just for
> that. WDYT?

We could change maintenance.git to not use the default mumi package but
to build the latest mumi at all times.  I won’t be able to do this,
though.  Toddler has a high fever and I barely sleep these days.

-- 
Ricardo

Message #22 received at 54024-done <at> debbugs.gnu.org (full text, mbox):

From: Arun Isaac <arunisaac <at> systemreboot.net>
To: Ricardo Wurmus <rekado <at> elephly.net>
Cc: 54024-done <at> debbugs.gnu.org
Subject: Re: [mumi] Preprocess search query from GraphQL before use
Date: Sun, 20 Feb 2022 01:49:10 +0530

[Message part 1 (text/plain, inline)]

Hi Ricardo,

> It’s not up because we’re still having problems with the migration.

No problem, this isn't too urgent.

>> Should we consider more automated ways to redeploy mumi on every commit?
>> Or maybe, we should run a separate development server just for
>> that. WDYT?
>
> We could change maintenance.git to not use the default mumi package but
> to build the latest mumi at all times.  I won’t be able to do this,
> though.  Toddler has a high fever and I barely sleep these days.

I'll take a look at this. Maybe, I can give this a shot. This may be
similar to a Guix based continuous deployment system I just set up at
work.

You take care, and take care of the little one. Family comes before
Guix.

Regards,
Arun

[signature.asc (application/pgp-signature, inline)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.