How to use ErrorOr...

[Jacrispys]

I am currently implementing a typesafe client for queries, when trying to add a ErrorOr handler to my class I get this error...

It seems that the ErrorOr is messing up the syntax, but I'm not sure what type parameter I'm supposed to pass as the "location" to the ErrorOr object... Any help is greatly appreciated!

[phillip-kruger]

@jmartisk ^^^ ?

[t1]

I wrote a minimalistic demo to show how the (test) GraphQL Client can read partial data via ErrorOr on a field: https://github.com/t1/error-or-demo

Maybe it's helpful. Otherwise, could you provide a reproducer, @Jacrispys ?

[Jacrispys]

I wrote a minimalistic demo to show how the (test) GraphQL Client can read partial data via ErrorOr on a field: https://github.com/t1/error-or-demo

Maybe it's helpful. Otherwise, could you provide a reproducer, @Jacrispys ?

This is helpful, though i'm trying to throw an AuthorizationException when the query is called if the user doesn't use the correct headers... Any idea how I would throw an error from the query itself and not one of the fields?

or maybe this is a better way of doing it and using and impl?

You shouldn't add an ErrorOr field into the class, you should wrap the class into ErrorOr in your client API, so the query method becomes

@Query("me")
ErrorOr<SelfUser> getSelfUser();

[Jacrispys]

I wrote a minimalistic demo to show how the (test) GraphQL Client can read partial data via ErrorOr on a field: https://github.com/t1/error-or-demo

Maybe it's helpful. Otherwise, could you provide a reproducer, @Jacrispys ?

This is helpful, though i'm trying to throw an AuthorizationException when the query is called if the user doesn't use the correct headers... Any idea how I would throw an error from the query itself and not one of the fields?

or maybe this is a better way of doing it and using and impl?

You shouldn't add an ErrorOr field into the class, you should wrap the class into ErrorOr in your client API, so the query method becomes

@query("me")

ErrorOr getSelfUser();

For more clarification, I would like to have the ClientApi interface to be implemented into a impl of the interface where I can throw errors and execute other methods... it would also be nice to be able to get the graphql errors from the ClientApi and throw my own more specific errors.

[Jacrispys]

Fixed with this code...

[t1]

To me, this inheritance is confusing. I'd rename the T4J interface to Api and the T4JImpl class to T4J and not let it implement the interface. The T4J is the only one using the Api, so I'd make it a nested interface in T4J, so it's T4J$Api. Just my 2¢.