Skip to content

Commit

Permalink
[deep link] Add more explanation for cross checking. (#8020)
Browse files Browse the repository at this point in the history
* wording

1

* Update validation_details_view.dart

* wording
  • Loading branch information
hangyujin authored Jul 11, 2024
1 parent 0fd50b0 commit 8935f28
Showing 1 changed file with 19 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -578,6 +578,7 @@ class _CrossCheckTable extends StatelessWidget {
final missingIos = hasIosAasaFile && !linkData.os.contains(PlatformOS.ios);
final missingAndroid =
hasAndroidAssetLinksFile && !linkData.os.contains(PlatformOS.android);
if (!missingIos && !missingAndroid) return const SizedBox.shrink();

final theme = Theme.of(context);
final domainMissing = Text(
Expand All @@ -602,14 +603,30 @@ class _CrossCheckTable extends StatelessWidget {
os: PlatformOS.android,
checkName: 'Manifest file',
status: domainMissing,
children: const <Widget>[],
children: const <Widget>[
Padding(
padding: EdgeInsets.symmetric(horizontal: defaultSpacing),
child: Text(
'This domain has an Asset link json file but it\'s missing in the android manifest file. '
'If you want to set up deep linking for Android, you need to add this domain '
'to your AndroidManifest.xml file.'),
),
],
),
if (missingIos)
_CheckExpansionTile(
os: PlatformOS.ios,
checkName: 'Settings',
status: domainMissing,
children: const <Widget>[],
children: const <Widget>[
Padding(
padding: EdgeInsets.symmetric(horizontal: defaultSpacing),
child: Text(
'This domain has an AASA file but it\'s missing in local settings. '
'If you want to set up deep linking for iOS, you need to add this domain '
'to your info.Plist file.'),
),
],
),
],
);
Expand Down

0 comments on commit 8935f28

Please sign in to comment.