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

fix: change request cookie type to be Record<string, string> #1857

Merged
merged 2 commits into from
Nov 16, 2023
Merged

fix: change request cookie type to be Record<string, string> #1857

merged 2 commits into from
Nov 16, 2023

Conversation

ezzatron
Copy link
Contributor

Request cookie value types are currently incorrectly typed as string | Array<string>. This PR changes the request cookie value type to a simple string.

fix #1854

Request cookie value types are currently incorrectly typed as
`string | Array<string>`. This commit changes the request cookie value
type to a simple `string`.

fix #1854
Copy link
Member

@kettanaito kettanaito left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me! Thanks, @ezzatron 👏

@kettanaito kettanaito changed the title fix: change request cookie value type to string fix: change request cookie type to be Record<string, string> Nov 15, 2023
@kettanaito
Copy link
Member

This is fantastic. Once again, great job, @ezzatron! Welcome to contributors! 🎉

@kettanaito kettanaito merged commit 75bbb96 into mswjs:main Nov 16, 2023
8 checks passed
@kettanaito
Copy link
Member

Released: v2.0.7 🎉

This has been released in v2.0.7!

Make sure to always update to the latest version (npm i msw@latest) to get the newest features and bug fixes.


Predictable release automation by @ossjs/release.

@ezzatron ezzatron deleted the fix/request-cookie-type branch November 16, 2023 23:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants