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(core): sdkVersion and osVersion type adjustments #10271

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

NathanWalker
Copy link
Contributor

PR Checklist

What is the current behavior?

Currently, both Device.sdkVersion and Device.osVersion return a number in Android. The typings state that this should be a string so expressions like if (Device.sdkVersion >= 23) {...} are not possible.

What is the new behavior?

I added number | string to the IDevice interface declaration so these expressions are allowed without the need to add @ts-ignore. I also updated the JSDoc to reflect this change.

These two return a number in Android (can't test iOS). Adding this fix would make expressions such as `Device.sdkVersion >= 23` possible without the need to add `@ts-ignore` before
Updated JSDoc to reflect Android behavior
@nx-cloud
Copy link

nx-cloud bot commented Apr 17, 2023

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 4cfc742. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch


✅ Successfully ran 2 targets

Sent with 💌 from NxCloud.

@cla-bot cla-bot bot added the cla: yes label Apr 17, 2023
@NathanWalker
Copy link
Contributor Author

See comments here

@NathanWalker NathanWalker added this to the 8.6 milestone Apr 17, 2023
@NathanWalker NathanWalker modified the milestones: 8.6, 9.0 Sep 29, 2023
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