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(hasNotch): add Redmi Note 8 Pro to hasNotch list #1276

Merged
merged 4 commits into from
Aug 5, 2021

Conversation

yevhenlv
Copy link
Contributor

@yevhenlv yevhenlv commented Aug 3, 2021

Description

Fixes #1275

Add Redmi Note 8 Pro to the list of devices with notch

Compatibility

OS Implemented
iOS
Android
Windows

Checklist

  • I have tested this on a device/simulator for each compatible OS
  • I added the documentation in README.md
  • I updated the typings files (privateTypes.ts, types.ts)
  • I added a sample use of the API (example/App.js)

Add Redmi Note 8 Pro
@yevhenlv yevhenlv changed the title Update devices with notch list feat: Update devices with notch list Aug 3, 2021
@yevhenlv yevhenlv changed the title feat: Update devices with notch list fix: Update devices with notch list Aug 3, 2021
CHANGELOG.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@mikehardy mikehardy left a comment

Choose a reason for hiding this comment

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

This was close - but you actually did more work than necessary! The changelog is updated automatically via semantic-release when a PR is merged. I proposed a change + committed same to remove that so it's just the notch list update - if it passes CI I'll merge it and it will auto-release

Thanks!

@mikehardy mikehardy changed the title fix: Update devices with notch list fix(hasNotch): add Redmi Note 8 Pro to hasNotch list Aug 4, 2021
@mikehardy mikehardy added the pending-merge looks good, waiting on CI or similar label Aug 4, 2021
@mikehardy mikehardy merged commit 06e2b6c into react-native-device-info:master Aug 5, 2021
@mikehardy mikehardy removed the pending-merge looks good, waiting on CI or similar label Aug 5, 2021
rndi-bot pushed a commit that referenced this pull request Aug 5, 2021
## [8.1.6](v8.1.5...v8.1.6) (2021-08-05)

### Bug Fixes

* **hasNotch:** add Redmi Note 8 Pro to hasNotch list ([#1276](#1276)) ([06e2b6c](06e2b6c))
@rndi-bot
Copy link
Collaborator

rndi-bot commented Aug 5, 2021

🎉 This PR is included in version 8.1.6 🎉

The release is available on:

Your semantic-release bot 📦🚀

WictorWilnd added a commit to WictorWilnd/react-native-device-info that referenced this pull request Feb 27, 2023
wdavis122 added a commit to wdavis122/react-native-device-info that referenced this pull request Jun 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Devices with notch - add new
3 participants