Orientation sensor

[gramotei]

Hello! I've added Orientation Sensor that returns rotation quaternion

[DanielMSchmidt]

Very nice, thank you for the PR and addition to the project 👍 I know this is something a lot of people have waited for for a long time. I just have a few comments on the docs, if these are resolved we can merge it 👍

[DanielMSchmidt]

Travis is red du to code formatting, npm run format should fix it

[gramotei]

@DanielMSchmidt yeah I see it's red because of formatting but when I run format it also changes format of other files. not sure if its ok :) I can do it for sure if you don't mind

        modified:   docs/Changelog.md
        modified:   docs/Usage.md
        modified:   examples/BasicSensors/App.js
        modified:   examples/BasicSensors/SensorView.js
        modified:   examples/GryoscopeImage/App.js
        modified:   examples/KeepTheBallGame/App.js
        modified:   examples/KeepTheBallGame/Ball.js
        modified:   examples/KeepTheBallGame/Game.js
        modified:   examples/KeepTheBallGame/GameOver.js
        modified:   tests/App.js
        modified:   tests/SensorView.js
        modified:   tests/e2e/init.js
        modified:   website/core/Footer.js
        modified:   website/i18n/en.json
        modified:   website/pages/en/index.js
        modified:   website/siteConfig.js

[DanielMSchmidt]

Thanks for the help 👍 Yeah I'll commit them right away, sorry that this fell on your PR :(

[kirkryan]

Hi all, I have version 7.0.2 and can see this was committed into react-native-sensors but it seems to be unavailable in the current code - did the fail above prevent the merge and if what are the next steps and how can I help?

[DanielMSchmidt]

It did not prevent the merge, we just forgot to change the d.ts files accordingly in this PR. Could you create a follow-up PR?

[gramotei]

Hm. But I added them https://github.com/react-native-sensors/react-native-sensors/blob/master/index.d.ts ?

[DanielMSchmidt]

Oh yeah, you are right, I wonder what the problem is 🤔

[kirkryan]

@gramotei @DanielMSchmidt - should we jump on a teams meeting and work it out? Would be great to get orientation in there! I'm on UK time, but happy to work around you guys.

[DanielMSchmidt]

I am in EU timezone as well, but I have a full time job that is not React Native Sensors, so I don't really have the time to jump on the call. I can see if I can devote some time to this issue over the weekend

[kirkryan]

Thanks, @DanielMSchmidt and totally understood!

[kirkryan]

@DanielMSchmidt - so, I was able to branch off - create my own npm package based on the current code and voila it works. It looks like it just needs the npm package building for the react-native-sensors repo.

[ebrahimhassan121]

@DanielMSchmidt
you can install the code on the master using npm i react-native-sensors/react-native-sensors.git

[DanielMSchmidt]

I think the deployment failed in general due to some strange error with prettier, I fixed the check now so that the tests should pass again, hope we can get a nice release out with all the features soonish

[kirkryan]

I can confirm that the code works great - I pulled to my own repo and built a GitHub npm repo to test - so looking forward to coming back into the main branch once it's updated!

[DanielMSchmidt]

🎉 This PR is included in version 7.1.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀

[arthedza]

@DanielMSchmidt @gramotei Please update the documentation: sensor description is absolutely absent there.
#409