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

Update types.ts to fix wrong void return type of multiGet #767

Merged
merged 3 commits into from
Mar 16, 2022

Conversation

mtt87
Copy link
Contributor

@mtt87 mtt87 commented Mar 9, 2022

Fix #764

Summary

Test Plan

@mtt87 mtt87 mentioned this pull request Mar 9, 2022
5 tasks
src/types.ts Show resolved Hide resolved
@krizzu
Copy link
Member

krizzu commented Mar 15, 2022

There's one last bit that needs to be one - updating the type of MultiRequest type.

-  resolve?: (result?: readonly KeyValuePair[]) => void;
+  resolve?: (result: readonly KeyValuePair[]) => void;

@tido64
Copy link
Member

tido64 commented Mar 15, 2022

Sorry I couldn't find time for this. I'm glad that you two managed to figure this out 😄

Copy link
Member

@krizzu krizzu left a comment

Choose a reason for hiding this comment

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

Great 👏

@krizzu krizzu merged commit 6ba9d69 into react-native-async-storage:master Mar 16, 2022
@krizzu
Copy link
Member

krizzu commented Mar 16, 2022

🎉 This PR is included in version 1.16.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

multiGet return type incorrect?
3 participants