poap-xyz / poap.js

POAP implemetation in javascript
https://sdk.poap.tech/
MIT License
9 stars 8 forks source link

Improve Compass types and errors #92

Closed jm42 closed 7 months ago

jm42 commented 7 months ago

Description

The compass error management was not good to understand what was happening under. It had some problems like doing a response.json when the status code is not 200, double-throw without needed in request/fetchGraphQL and the catch was for everything instead of more fine-grained problems. Now there is a custom error that will return GraphQL errors. Also, changed the use of any for unknown and remove the ts-ignore.

Type of change

Checklist

netlify[bot] commented 7 months ago

Deploy Preview for eloquent-alpaca-5d321f canceled.

Name Link
Latest commit 2a01abcd5a7da0c2f95f5d9f7a097853bf9070cb
Latest deploy log https://app.netlify.com/sites/eloquent-alpaca-5d321f/deploys/65f8065600c97e0008429d12