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

feat(fake-browser): Upgrade webextension-polyfill types to 0.10.5 #46

Conversation

kayac-chang
Copy link
Contributor

@kayac-chang kayac-chang commented Nov 1, 2023

solved #45

@kayac-chang
Copy link
Contributor Author

Hello @aklinker1.
Since the session storage needs to raise the version of @types/webextension-polyfill to 0.10.5,
we also need to mock the new methods unrelated to the session storage, otherwise caused the type check to fail.
Should I do that in this PR?

Copy link
Owner

@aklinker1 aklinker1 left a comment

Choose a reason for hiding this comment

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

Thanks for opening a PR!

Since the session storage needs to raise the version of @types/webextension-polyfill to 0.10.5,
we also need to mock the new methods unrelated to the session storage, otherwise caused the type check to fail.
Should I do that in this PR?

Yeah, we should do that in this PR as well. I updated the PR title to reflect this.

packages/fake-browser/src/apis/storage.ts Outdated Show resolved Hide resolved
packages/fake-browser/src/apis/storage.test.ts Outdated Show resolved Hide resolved
packages/fake-browser/src/apis/storage.test.ts Outdated Show resolved Hide resolved
@aklinker1 aklinker1 changed the title feat: 🎸 fake browser support session storage feat(fake-browser): Upgrade webextension-polyfill types to 0.10.5 Nov 2, 2023
Copy link
Owner

@aklinker1 aklinker1 left a comment

Choose a reason for hiding this comment

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

Awesome, thanks for making this. I'll get it merged and released once I get the checks passing 👍

@aklinker1 aklinker1 merged commit cbfc4e5 into aklinker1:main Nov 3, 2023
1 check passed
@kayac-chang kayac-chang deleted the feat/fake-browser-support-session-storage branch November 3, 2023 16:13
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.

2 participants