[PATCH] web: Include blocked-by in graphql <issue>.

[PATCH] web: Include blocked-by in graphql <issue>.

[Christopher Baines]

* mumi/web/graphql.scm (<issue>): Include blocked-by.
---
 mumi/web/graphql.scm | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/mumi/web/graphql.scm b/mumi/web/graphql.scm
index aa4f38c..3cc19cd 100644
--- a/mumi/web/graphql.scm
+++ b/mumi/web/graphql.scm
@@ -63,7 +63,10 @@
               (list))))
   (messages (non-nullable-type (list-type <email>))
             (lambda (parent . _)
-              (issue-messages (bug-num parent)))))
+              (issue-messages (bug-num parent))))
+  (blocked-by (non-nullable-type (list-type <issue>))
+              (lambda (parent . _)
+                (map bug-status (bug-blockedby parent)))))
 
 (define-object-type <person>
   (name <string> (lambda (parent . _)
-- 
2.40.1

Re: [PATCH] web: Include blocked-by in graphql <issue>.

[Christopher Baines]

[-- Attachment #1: Type: text/plain, Size: 972 bytes --]


Christopher Baines <mail@cbaines.net> writes:

> * mumi/web/graphql.scm (<issue>): Include blocked-by.
> ---
>  mumi/web/graphql.scm | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/mumi/web/graphql.scm b/mumi/web/graphql.scm
> index aa4f38c..3cc19cd 100644
> --- a/mumi/web/graphql.scm
> +++ b/mumi/web/graphql.scm
> @@ -63,7 +63,10 @@
>                (list))))
>    (messages (non-nullable-type (list-type <email>))
>              (lambda (parent . _)
> -              (issue-messages (bug-num parent)))))
> +              (issue-messages (bug-num parent))))
> +  (blocked-by (non-nullable-type (list-type <issue>))
> +              (lambda (parent . _)
> +                (map bug-status (bug-blockedby parent)))))
>  
>  (define-object-type <person>
>    (name <string> (lambda (parent . _)

I'm looking at this so that the qa-frontpage can see what issues are
blocked.

I'm not sure how to test this though.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 987 bytes --]

Re: [PATCH] web: Include blocked-by in graphql <issue>.

[Arun Isaac]

Hi Chris,

Thanks for the patch! I have pushed it after renaming blocked-by to
blocked_by. Underscores are a bit cringey, but unfortunately GraphQL
doesn't like hyphens.

To see the effects of this change, we need to reconfigure berlin to run
the latest mumi. I will figure that out and do it soon.

Regards,
Arun

Re: [PATCH] web: Include blocked-by in graphql <issue>.

[Christopher Baines]

[-- Attachment #1: Type: text/plain, Size: 411 bytes --]


Arun Isaac <arunisaac@systemreboot.net> writes:

> Thanks for the patch! I have pushed it after renaming blocked-by to
> blocked_by. Underscores are a bit cringey, but unfortunately GraphQL
> doesn't like hyphens.
>
> To see the effects of this change, we need to reconfigure berlin to run
> the latest mumi. I will figure that out and do it soon.

Awesome, thanks for fixing my rough untested changes!

Chris

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 987 bytes --]