-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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(getDeviceName): un-memoize device name so it updates #1332
Conversation
removes memoization of device name because the device's name can be changed resolves #1272 Co-authored-by: @chrisdrackett
moves function from `memoizedStringGetters` to `nonMemoizedStringGetters`
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! Thanks for fixing this. Should be a squash-merge since the test is in a separate commit from the fix, otherwise master won't be git bisectable as it'll fail testing on the first commit without the second
Please feel free to merge as soon as CI looks reasonable - in fact it's just iOS Android on-device stuff now and the jest already passed so this is good to go IMHO
🎉 This PR is included in version 8.4.3 🎉 The release is available on: Your semantic-release bot 📦🚀 |
## [8.4.3](react-native-device-info/react-native-device-info@v8.4.2...v8.4.3) (2021-10-26) ### Bug Fixes * **getDeviceName:** un-memoize device name so it updates ([#1332](react-native-device-info/react-native-device-info#1332)) ([67b7155](react-native-device-info/react-native-device-info@67b7155)), closes [#1272](react-native-device-info/react-native-device-info#1272)
## [8.4.3](react-native-device-info/react-native-device-info@v8.4.2...v8.4.3) (2021-10-26) ### Bug Fixes * **getDeviceName:** un-memoize device name so it updates ([#1332](react-native-device-info/react-native-device-info#1332)) ([7026844](react-native-device-info/react-native-device-info@7026844)), closes [#1272](react-native-device-info/react-native-device-info#1272)
getDeviceName
getDeviceName
Description
Fixes #1272
all real credit goes to @chrisdrackett for noticing this issue and the initial PR.
Compatibility
Checklist
README.md
privateTypes.ts
,types.ts
)example/App.js
)