Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

More dev friendly errors #21

Closed
vikiival opened this issue Mar 17, 2023 · 4 comments · Fixed by #38
Closed

More dev friendly errors #21

vikiival opened this issue Mar 17, 2023 · 4 comments · Fixed by #38
Milestone

Comments

@vikiival
Copy link
Member

It would be nicer to throw away errors that are more user friendly

This one could be "unable to perform fetch for non-existing indexer, add parameter prefix when calling getClient"

image

ht/ @poppyseedDev

@roiLeo
Copy link
Contributor

roiLeo commented Mar 29, 2023

Indexer not found when no client set, shoudl fallback to smething default

@vikiival
Copy link
Member Author

shoudl fallback to smething default

and It would return undefined if you provide bad params.

We can either implement:

@roiLeo
Copy link
Contributor

roiLeo commented Mar 29, 2023

or add it to readme (or jsdoc) so devs won't loose time debuging this one

@vikiival
Copy link
Member Author

or add it to readme (or jsdoc) so devs won't loose time debuging this one

added 1764cf0

@vikiival vikiival added this to the Sub1 milestone Sep 3, 2023
vikiival added a commit that referenced this issue Sep 3, 2023
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 a pull request may close this issue.

2 participants