-
Notifications
You must be signed in to change notification settings - Fork 24.8k
Open
Labels
Needs: AttentionIssues where the author has responded to feedback.Issues where the author has responded to feedback.Needs: ReproThis issue could be improved with a clear list of steps to reproduce the issue.This issue could be improved with a clear list of steps to reproduce the issue.Resolution: PR SubmittedA pull request with a fix has been provided.A pull request with a fix has been provided.
Description
According to the type definition, Platform.Version
is always defined and is either a string or a number.
Version: number | string; |
However, in the Web platform it is undefined
.
react-native/packages/react-native/Libraries/Utilities/PlatformTypes.js
Lines 162 to 166 in ce306ac
type WebPlatform = { | |
OS: 'web', | |
// $FlowFixMe[unsafe-getters-setters] | |
get Version(): void, | |
// $FlowFixMe[unsafe-getters-setters] |
Please either set it optional or ensure the web version returns a sensible Version
Metadata
Metadata
Assignees
Labels
Needs: AttentionIssues where the author has responded to feedback.Issues where the author has responded to feedback.Needs: ReproThis issue could be improved with a clear list of steps to reproduce the issue.This issue could be improved with a clear list of steps to reproduce the issue.Resolution: PR SubmittedA pull request with a fix has been provided.A pull request with a fix has been provided.