-
Notifications
You must be signed in to change notification settings - Fork 4
chore: maintain package #19
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
Conversation
|
TODO: Consider turning |
angeloashmore
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! I left a question about an old error message, not related to the changes in this PR.
src/channel/messages.ts
Outdated
| ): ErrorResponseMessage<TError> => { | ||
| if (status < 400) { | ||
| throw new TypeError( | ||
| `Invalid error status code, expected status to be within \`[500;600[\`, got \`${status}\``, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this error message contain [400, 600), not [500;600[?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice catch!
Since |
Co-authored-by: Angelo Ashmore <angeloashmore@users.noreply.github.com>
Resolves: https://prismic-team.slack.com/archives/CPG31MDL1/p1767962753033489
Related:
<SliceSimulator>component prismic-next#120<SliceSimulator>component prismic-vue#90<SliceSimulator>component nuxt-modules/prismic#237<SliceSimulator>component prismic-svelte#40Description
statusesCJS dependencytsdownfor building the package)Checklist
Preview
How to QA 1
Footnotes
Please use these labels when submitting a review:
⚠️ #issue: Strongly suggest a change.
❓ #ask: Ask a question.
💡 #idea: Suggest an idea.
🎉 #nice: Share a compliment. ↩