Skip to content

Add support for onError#6860

Merged
martinbonnin merged 1 commit intomainfrom
on-error
Jan 29, 2026
Merged

Add support for onError#6860
martinbonnin merged 1 commit intomainfrom
on-error

Conversation

@martinbonnin
Copy link
Copy Markdown
Contributor

@martinbonnin martinbonnin requested a review from BoD as a code owner January 28, 2026 17:10
@apollo-librarian
Copy link
Copy Markdown

apollo-librarian bot commented Jan 28, 2026

✅ Docs preview has no changes

The preview was not built because there were no changes.

Build ID: 51df5d1a7a793fb13cc13022
Build Logs: View logs

public fun getChildren ()Ljava/util/List;
public final fun getDescription ()Ljava/lang/String;
public final fun getQualifiedName ()Ljava/lang/String;
public final fun getName ()Ljava/lang/String;
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm advocating for name instead of identifier because there is name in many other places and I don't think we need specific language for this.

See graphql/graphql-spec#1208 (comment)

public static final fun parseAsGraphQLRequest (Ljava/util/Map;)Ljava/lang/Object;
public static final fun parseAsGraphQLRequest (Lokio/BufferedSource;)Ljava/lang/Object;
public static final fun toExternalValueMap (Ljava/util/Map;)Ljava/lang/Object;
public static final fun toGraphQLRequest (Ljava/lang/String;)Lcom/apollographql/apollo/execution/GraphQLRequest;
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sneaky removal before it's too late. 🥷

Copy link
Copy Markdown
Contributor

@BoD BoD left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@martinbonnin martinbonnin merged commit df1bf18 into main Jan 29, 2026
7 checks passed
@martinbonnin martinbonnin deleted the on-error branch January 29, 2026 09:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants