Error when using relay module without Django

[blb-ventures]

Error when using relay module without Django

[bellini666]

@sloria hrm, I'll test your example in the next couple of days and try to fix the relay code that is giving those issues!

[bellini666]

Hey @sloria ,

I have to be honest with you that, although I wrote that relay module specifically not attached to django to allow it to be used standalone, I mostly tested it with django =P

One thing you could do to workaround that is to add an _id to Fruit and define a resolve_id for it. Like this:

@strawberry.type
class Fruit(relay.Node):
    _id: strawberry.Private[str]
    name: str
    description: Optional[str]

    @classmethod
    def resolve_id(cls, root, *, info=None):
        return root._id

By typing it as Private it should not be exposed to graphql, and the resolve_id method should use it.

I'll try to think how to allow id to be passed like you did while still being able to have the custom resolver for it.

[sloria]

Ah I see. Unfortunately, I'm still unable to get id to properly resolve

from typing import Iterable, Optional

import strawberry
from strawberry.types.info import Info

from .utils import relay

fruits = [
    {
        "_id": 1,
        "name": "Banana",
        "description": "Lorem ipsum",
    },
    {
        "_id": 2,
        "name": "Apple",
        "description": None,
    },
    {
        "_id": 3,
        "name": "Orange",
        "description": "Lorem ipsum",
    },
]


@strawberry.type
class Fruit(relay.Node):
    _id: strawberry.Private[str]
    name: str
    description: Optional[str]

    @classmethod
    def resolve_id(cls, root, *, info=None):
        return str(root._id)

    @classmethod
    def resolve_node(
        cls,
        node_id: str,
        *,
        info: Optional[Info] = None,
        required: bool = False,
    ):
        for fruit in fruits:
            if str(fruit["_id"]) == node_id:
                return Fruit(**fruit)

        if required:
            raise ValueError(f"Fruit by id {node_id} not found.")

        return None

    @classmethod
    def resolve_nodes(
        cls,
        *,
        info: Optional[Info] = None,
        node_ids: Optional[Iterable[str]] = None,
    ):
        node_ids = node_ids and set(node_ids)

        for fruit in fruits:
            if node_ids is not None and str(fruit["id"]) not in node_ids:
                continue

            yield Fruit(**fruit)


@strawberry.type
class Query:
    fruit: Fruit = relay.node()
    fruits_conn: relay.Connection[Fruit] = relay.connection()

    @relay.connection
    def fruits_conn_with_filter(self, name_startswith: str) -> Iterable[Fruit]:
        for fruit in fruits:
            if fruit["name"].startswith(name_startswith):
                yield Fruit(**fruit)


@strawberry.type
class Mutation:
    @relay.input_mutation
    def create_fruit(self, name: str, description: Optional[str]) -> Fruit:
        fruit_data = {
            "id": max(f["id"] for f in fruits) + 1,
            "name": name,
            "description": description,
        }
        fruits.append(fruit_data)
        return Fruit(**fruit_data)


schema = strawberry.Schema(query=Query, mutation=Mutation)

If I query for a fruit's ID...

query {
    fruit(id: "RnJ1aXQ6MQ==") {
        id
        name
    }
}

I get

utils.relay.GlobalIDValueError: node_id is expected to be a string, found <bound method Node.id of <class 'utils.relay.Node'>>