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

bugfix: handle if errors is not an array #1082

Conversation

faizplus
Copy link

closes #1024

  • check that error.originalError.errors is an array before mapping it.
  • if not an array, but is defined, try to handle it anyway.

closes mercurius-js#1024

check that error.originalError.errors is an array before mapping it.

if not an array, but is defined, try to handle it anyway.
@faizplus faizplus marked this pull request as ready for review February 13, 2024 12:45
@faizplus faizplus closed this Feb 13, 2024
@faizplus faizplus deleted the faizplus-bugfix/defaultErrorFormatter-handle-non-array-errors branch February 13, 2024 13:19
@faizplus faizplus restored the faizplus-bugfix/defaultErrorFormatter-handle-non-array-errors branch February 13, 2024 13:21
@faizplus faizplus deleted the faizplus-bugfix/defaultErrorFormatter-handle-non-array-errors branch February 26, 2024 12:51
@mike-van
Copy link

mike-van commented Mar 5, 2024

Hi im encountering this error, is this not merged yet? Is there anything I could do to help get this merge?

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.

Occasional "error.originalError.errors.map is not a function" errors
2 participants