Skip to content

fix: simulator detection in demo apps#1125

Merged
msluszniak merged 4 commits intomainfrom
@chmjkb/fix-cv-demo-app
May 8, 2026
Merged

fix: simulator detection in demo apps#1125
msluszniak merged 4 commits intomainfrom
@chmjkb/fix-cv-demo-app

Conversation

@chmjkb
Copy link
Copy Markdown
Collaborator

@chmjkb chmjkb commented May 7, 2026

Description

Due to deprecation of the used variable from Constants (therefore introducing whong behaviour on physical devices), this PR uses react-native-device-info for simulator detecting, instead of Expo Constants.

Introduces a breaking change?

  • Yes
  • No

Type of change

  • Bug fix (change which fixes an issue)
  • New feature (change which adds functionality)
  • Documentation update (improves or adds clarity to existing documentation)
  • Other (chores, tests, code style improvements etc.)

Tested on

  • iOS
  • Android

Testing instructions

Screenshots

Related issues

Checklist

  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have updated the documentation accordingly
  • My changes generate no new warnings

Additional notes

@chmjkb chmjkb marked this pull request as ready for review May 8, 2026 06:07
@chmjkb chmjkb requested a review from benITo47 May 8, 2026 06:07
@chmjkb chmjkb self-assigned this May 8, 2026
@chmjkb chmjkb added demo app bug fix PRs that are fixing bugs labels May 8, 2026
@msluszniak msluszniak self-requested a review May 8, 2026 07:58
@msluszniak msluszniak merged commit 0c64923 into main May 8, 2026
4 checks passed
@msluszniak msluszniak deleted the @chmjkb/fix-cv-demo-app branch May 8, 2026 08:41
IgorSwat pushed a commit that referenced this pull request May 11, 2026
## Description

Due to deprecation of the used variable from Constants (therefore
introducing whong behaviour on physical devices), this PR uses
`react-native-device-info` for simulator detecting, instead of Expo
Constants.

### Introduces a breaking change?

- [ ] Yes
- [x] No

### Type of change

- [x] Bug fix (change which fixes an issue)
- [ ] New feature (change which adds functionality)
- [ ] Documentation update (improves or adds clarity to existing
documentation)
- [ ] Other (chores, tests, code style improvements etc.)

### Tested on

- [x] iOS
- [x] Android

### Testing instructions

<!-- Provide step-by-step instructions on how to test your changes.
Include setup details if necessary. -->

### Screenshots

<!-- Add screenshots here, if applicable -->

### Related issues

<!-- Link related issues here using #issue-number -->

### Checklist

- [ ] I have performed a self-review of my code
- [ ] I have commented my code, particularly in hard-to-understand areas
- [ ] I have updated the documentation accordingly
- [ ] My changes generate no new warnings

### Additional notes

<!-- Include any additional information, assumptions, or context that
reviewers might need to understand this PR. -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug fix PRs that are fixing bugs demo app

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants