-
Notifications
You must be signed in to change notification settings - Fork 4k
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
docs(app_check): add additional steps to ensure App Check works on specific devices #17110
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've made some suggestions 👍
docs/app-check/default-providers.md
Outdated
@@ -141,3 +141,8 @@ environment, you can create a debug build of your app that uses the | |||
App Check debug provider instead of a real attestation provider. | |||
|
|||
See [Use App Check with the debug provider in Flutter apps](/docs/app-check/flutter/debug-provider). | |||
|
|||
### Additional Steps to Enable App Check on Specific Devices (e.g. Xiaomi) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
### Additional Steps to Enable App Check on Specific Devices (e.g. Xiaomi) |
docs/app-check/default-providers.md
Outdated
@@ -141,3 +141,8 @@ environment, you can create a debug build of your app that uses the | |||
App Check debug provider instead of a real attestation provider. | |||
|
|||
See [Use App Check with the debug provider in Flutter apps](/docs/app-check/flutter/debug-provider). | |||
|
|||
### Additional Steps to Enable App Check on Specific Devices (e.g. Xiaomi) | |||
For certain devices like Xiaomi, you need to enable "Meets basic device integrity" in the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For certain devices like Xiaomi, you need to enable "Meets basic device integrity" in the | |
Note: For certain Android devices, you need to enable "Meets basic device integrity" in the |
docs/app-check/default-providers.md
Outdated
|
||
### Additional Steps to Enable App Check on Specific Devices (e.g. Xiaomi) | ||
For certain devices like Xiaomi, you need to enable "Meets basic device integrity" in the | ||
Play Integrity API settings, as mentioned in [this issue comment](https://github.com/firebase/flutterfire/issues/17106#issuecomment-2663422877), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Play Integrity API settings, as mentioned in [this issue comment](https://github.com/firebase/flutterfire/issues/17106#issuecomment-2663422877), | |
Play console |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That ought to do it 😄
docs/app-check/default-providers.md
Outdated
Note: For certain Android devices, you need to enable "Meets basic device integrity" in the | ||
Play console. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note: For certain Android devices, you need to enable "Meets basic device integrity" in the | |
Play console. | |
Note: For certain Android devices, you may need to enable "Meets basic device integrity" in the | |
Google Play console. |
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes (
[x]
).This will ensure a smooth and quick review process. Updating the
pubspec.yaml
and changelogs is not required.///
).melos run analyze
) does not report any problems on my PR.Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?