Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Remove < iOS 11 fallback API calls in openSettings: #1653

Merged
merged 4 commits into from
Mar 17, 2025

Conversation

jmagman
Copy link
Contributor

@jmagman jmagman commented Mar 7, 2025

geolocator_apple runs on a minimum of iOS 11

This means that if (@available(iOS 10, *)) will never be false. Remove the dead fallback. See my explanation in #1652 (comment) why I'm limiting the availability cleanup to this one -[UIApplication openURL:] method.

Remove fallbacks in tests as well.

On this PR:

Test Case '-[GeolocatorPluginTests testOpenAppSettings]' started.
Test Case '-[GeolocatorPluginTests testOpenAppSettings]' passed (0.001 seconds).

Part of #1652

Pre-launch Checklist

  • I made sure the project builds.
  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy, or this PR is does not need version changes.
  • I updated CHANGELOG.md to add a description of the change.
  • I updated/added relevant documentation (doc comments with ///).
  • I rebased onto main.
  • I added new tests to check the change I am making, or this PR does not need tests.
  • I made sure all existing and new tests are passing.
  • I ran dart format . and committed any changes.
  • I ran flutter analyze and fixed any errors.

@jmagman
Copy link
Contributor Author

jmagman commented Mar 7, 2025

Note this PR is easier to read with whitespace ignored:
Screenshot 2025-03-07 at 3 39 22 PM

@@ -1,3 +1,7 @@
## NEXT
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Happy to bump the version if you prefer, up to you!

Copy link
Contributor

Choose a reason for hiding this comment

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

@jmagman would be great. Thanks. Since I have not released 2.3.10 yet, I prefer it to be added to 2.3.10, since that is just a relative minor (update) release.

@jmagman jmagman marked this pull request as ready for review March 7, 2025 23:48
@TimHoogstrate
Copy link
Contributor

@jmagman, When I check the gelocator_apple package It looks like i see some more @available(iOS below or 11) Could we not clean it all?

Kind regards,

@mvanbeusekom
Copy link
Member

@jmagman, When I check the gelocator_apple package It looks like i see some more @available(iOS below or 11) Could we not clean it all?

Kind regards,

I think that would be a good idea, however maybe we can do so in a separate PR and start releasing this so the Flutter team can continue there work on the linter rule.

@TimHoogstrate TimHoogstrate merged commit abb74d2 into Baseflow:main Mar 17, 2025
1 check passed
@TimHoogstrate
Copy link
Contributor

@jmagman,

I've updated the version number and merged your changes. Thanks for your help. Your support is truly appreciated.

Kind regards,

@jmagman
Copy link
Contributor Author

jmagman commented Mar 17, 2025

Thank you @TimHoogstrate for handling that!

@jmagman jmagman deleted the openURL branch March 17, 2025 19:37
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants