Skip to content
This repository has been archived by the owner on Apr 11, 2024. It is now read-only.

Expand objects in HTTP request error messages #252

Merged
merged 1 commit into from
Oct 22, 2021

Conversation

paulomarg
Copy link
Contributor

WHY are these changes introduced?

Fixes #220
Fixes #237
Fixes #250

We can sometimes receive error messages that are not strings, and we were not printing them properly before.

WHAT is this pull request doing?

Printing the error messages as JSON so that we can handle any type of error.

Type of change

  • Patch: Bug (non-breaking change which fixes an issue)
  • Minor: New feature (non-breaking change which adds functionality)
  • Major: Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

  • I have added a changelog entry, prefixed by the type of change noted above
  • I have added/updated tests for this change
  • I have documented new APIs/updated the documentation for modified APIs (for public APIs)

@paulomarg paulomarg requested a review from a team as a code owner October 22, 2021 17:26
@paulomarg paulomarg merged commit c9d9451 into main Oct 22, 2021
@paulomarg paulomarg deleted the fix_object_error_messages branch October 22, 2021 17:31
@shopify-shipit shopify-shipit bot temporarily deployed to production October 22, 2021 17:36 Inactive
@shopify-shipit shopify-shipit bot temporarily deployed to next March 8, 2022 22:13 Inactive
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants