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

[Request]: Please add barometer #955

Closed
vitalsh opened this issue Jul 12, 2022 · 5 comments
Closed

[Request]: Please add barometer #955

vitalsh opened this issue Jul 12, 2022 · 5 comments
Labels
enhancement New feature or request Hacktoberfest Issues taking part in Hacktoberfest triage

Comments

@vitalsh
Copy link

vitalsh commented Jul 12, 2022

Plugin

sensors_plus

Use case

I want to make a cross-platform baro altimeter feature in the app I'm building (for gliding)

Proposal

Barometric readings: hectopascals and altitude

@vitalsh vitalsh added enhancement New feature or request triage labels Jul 12, 2022
@vitalsh vitalsh changed the title [Request]: Please ass barometer [Request]: Please add barometer Jul 12, 2022
@mhadaily mhadaily added the Hacktoberfest Issues taking part in Hacktoberfest label Sep 28, 2022
@aanas-sayed
Copy link
Contributor

Is there anyone working on this? I can try and push and update for the barometer on Android but as I understand unless all platforms are supported its unlikely to be accepted?

@miquelbeltran
Copy link
Member

That's correct, we don't accept new feature implementations if they only cover one platform in order to avoid feature fragmentation: https://github.com/fluttercommunity/plus_plugins/blob/main/CONTRIBUTING.md#-cannot-be-accepted

Nevertheless, there are several barometer plugins already: https://pub.dev/packages?q=barometer

@aanas-sayed
Copy link
Contributor

aanas-sayed commented Jul 8, 2024

Thanks @miquelbeltran for clarifying. I have implemented it on all platforms now and created a PR. I understand other plugins exist but there's none that are properly maintained. There's also none that integrate motion related sensors with altitude.

@aanas-sayed
Copy link
Contributor

Can this be closed now?

@miquelbeltran
Copy link
Member

Should have been closed when the PR was merged, indeed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Hacktoberfest Issues taking part in Hacktoberfest triage
Projects
None yet
Development

No branches or pull requests

4 participants