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 iPhone 13 lineup to list of devices with notch #1307

Merged
merged 1 commit into from
Sep 15, 2021

Conversation

rolfb
Copy link
Contributor

@rolfb rolfb commented Sep 15, 2021

Description

iPhone 13 lineup is about a week out, so adding this to the device list for devices with notch.

Compatibility

OS Implemented
iOS
Android
Windows

@rolfb rolfb changed the title Add iPhone 13 lineup to list of devices with notch chore(hasNotch): Add iPhone 13 lineup to list of devices with notch Sep 15, 2021
@rolfb
Copy link
Contributor Author

rolfb commented Sep 15, 2021

Just out of curiosity, what is following Conventional Commits useful for? I sense a learning opportunity.

@schie
Copy link
Contributor

schie commented Sep 15, 2021

Just out of curiosity, what is following Conventional Commits useful for? I sense a learning opportunity.

@rolfb checkout commitizen/cz-cli

@mikehardy mikehardy changed the title chore(hasNotch): Add iPhone 13 lineup to list of devices with notch fix(hasNotch): Add iPhone 13 lineup to list of devices with notch Sep 15, 2021
@mikehardy mikehardy added the pending-merge looks good, waiting on CI or similar label Sep 15, 2021
@mikehardy mikehardy merged commit 44193d5 into react-native-device-info:master Sep 15, 2021
@mikehardy mikehardy removed the pending-merge looks good, waiting on CI or similar label Sep 15, 2021
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.

Nice! Thanks

Conventional Commits, when followed strictly (a thing we enforce with a check here as you have found :-) ), allow you to infer the type of semantic version release (patch, feature, or breaking change) that is represented by a group of commits, which then allows you to automate releases completely when PRs are merged, via a github action that scans commit logs for conventional commits, infers the type of release and makes the correct version bump

You'll see it in action here - as soon as I merge this it'll do a patch release

rndi-bot pushed a commit that referenced this pull request Sep 15, 2021
## [8.3.2](v8.3.1...v8.3.2) (2021-09-15)

### Bug Fixes

* **hasNotch:** Add iPhone 13 lineup to list of devices with notch ([#1307](#1307)) ([44193d5](44193d5))
@rndi-bot
Copy link
Collaborator

🎉 This PR is included in version 8.3.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

@rolfb rolfb deleted the patch-1 branch September 16, 2021 06:47
@rolfb
Copy link
Contributor Author

rolfb commented Sep 16, 2021

@mikehardy Appreciate the explanation, and thank you for the quick merge and release :)

@hiteshSavkare
Copy link

hiteshSavkare commented Sep 16, 2021

Hi @mikehardy && @rolfb
Just curious to know don't we require the IOS file changes for notch support?
actually i am testing it on IOS 13 but always returning me false

I just found the reference here how it was for IOS 12, there i can see IOS file changes
https://github.com/react-native-device-info/react-native-device-info/pull/1099/files

@rolfb
Copy link
Contributor Author

rolfb commented Sep 16, 2021

Seems like I missed a file 🥺

@hiteshSavkare
Copy link

hiteshSavkare commented Sep 16, 2021

Seems like I missed a file 🥺

seems like that, hope you will add that file soon, eagerly waiting for testing , thanks for your efforts

rolfb added a commit to rolfb/react-native-device-info that referenced this pull request Sep 16, 2021
Adds missing device name by code for iPhone 13 lineup. Relates to react-native-device-info#1307
@rolfb
Copy link
Contributor Author

rolfb commented Sep 16, 2021

@hiteshSavkare can you have a look at #1309 and see if that fixes things for you?

@hiteshSavkare
Copy link

@hiteshSavkare can you have a look at #1309 and see if that fixes things for you?

@rolfb yes perfect, working as expected

@rolfb
Copy link
Contributor Author

rolfb commented Sep 16, 2021

@rolfb yes perfect, working as expected

@hiteshSavkare great! if you have a chance to verify that all four iPhone 13 devices are correctly mapped now I'd appreciate it 😊 I couldn't find official docs for firmware identifiers but it's highly likely that they are following the same pattern.

mikehardy pushed a commit that referenced this pull request Sep 16, 2021
* fix(hasNotch): Missing device name for iPhone 13

Adds missing device name by code for iPhone 13 lineup. Relates to #1307
Updated device code to name mapping to match the information in the sqlite3 device traits database provided with Xcode RC.
rndi-bot pushed a commit that referenced this pull request Sep 16, 2021
## [8.3.3](v8.3.2...v8.3.3) (2021-09-16)

### Bug Fixes

* **hasNotch:** Missing device name for iPhone 13 ([#1309](#1309)) ([0bc979e](0bc979e)), closes [#1307](#1307)
WictorWilnd added a commit to WictorWilnd/react-native-device-info that referenced this pull request Feb 27, 2023
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
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.

None yet

5 participants