-
-
Notifications
You must be signed in to change notification settings - Fork 869
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
Handle contacts limited permission #1395
base: main
Are you sure you want to change the base?
Conversation
The check failure of invoking prints seems to be from an old commit, not from my changes. Happy to remove them if required though |
Any update on when this would be merged? This looks good to me! |
Any update on this? I have same issue with iOS 18 |
Is there any update on this? As we have a client request for this feature. |
Any update ? Apple is deploying the 18.1 right now, I see customers with this issue |
for those who want it before merging
|
What about Android 14? it is directly show granted status |
Better to file a separate issue. This PR doesnt touch android |
The package wasn't currently handling the new
CNAuthorizationStatusLimited
introduce in ios18. Older versions of ios will remain unchanged. Here's a screen recording showcasing the changeSimulator.Screen.Recording.-.iPhone.14.Pro.-.2024-10-10.at.13.43.12.mp4
Simulator.Screen.Recording.-.Logged.In.16.Pro.-.2024-10-10.at.13.46.09.mp4
List at least one fixed issue.
Pre-launch Checklist
pubspec.yaml
with an appropriate new version according to the pub versioning philosophy, or this PR is does not need version changes.CHANGELOG.md
to add a description of the change.///
).main
.dart format .
and committed any changes.flutter analyze
and fixed any errors.