fix: standardize app package name to app.timesafari.app
Update package name consistently across all configurations to fix deep linking and app identification issues. Changes: - Update appId in package.json build config from "app.timesafari" to "app.timesafari.app" - Update force-stop commands in test scripts to use correct package name - Update URL scheme registration in iOS Info.plist to use consistent bundle ID - Update app identifier fallback values in iOS test script - Update serve script to use production mode with host flag Technical Details: - Standardized package name to "app.timesafari.app" across all configurations - Fixed ADB commands in test scripts to use correct package identifier - Updated iOS URL scheme registration to match Android package name - Added proper NODE_ENV and host configuration for serve command Benefits: - Fixes deep linking functionality by using consistent package identifiers - Ensures correct app identification across Android and iOS platforms - Improves test reliability with correct package references - Enables proper cross-platform URL scheme handling
This commit is contained in:
@@ -170,7 +170,7 @@ const executeDeeplink = async (url, description, log) => {
|
||||
|
||||
try {
|
||||
// Stop the app before executing the deep link
|
||||
execSync('adb shell am force-stop app.timesafari');
|
||||
execSync('adb shell am force-stop app.timesafari.app');
|
||||
await new Promise(resolve => setTimeout(resolve, 1000)); // Wait 1s
|
||||
|
||||
execSync(`adb shell am start -W -a android.intent.action.VIEW -d "${url}" -c android.intent.category.BROWSABLE`);
|
||||
@@ -207,10 +207,6 @@ const runDeeplinkTests = async (log) => {
|
||||
|
||||
// Test URLs
|
||||
const deeplinkTests = [
|
||||
{
|
||||
url: `timesafari://claim/${claimDetails.claim_id}`,
|
||||
description: 'Claim view'
|
||||
},
|
||||
{
|
||||
url: `timesafari://claim-cert/${claimDetails.claim_id}`,
|
||||
description: 'Claim certificate view'
|
||||
|
||||
@@ -800,7 +800,7 @@ const checkAndRegisterUrlScheme = (log) => {
|
||||
<array>
|
||||
<dict>
|
||||
<key>CFBundleURLName</key>
|
||||
<string>app.timesafari</string>
|
||||
<string>app.timesafari.app</string>
|
||||
<key>CFBundleURLSchemes</key>
|
||||
<array>
|
||||
<string>timesafari</string>
|
||||
@@ -855,10 +855,10 @@ const getAppIdentifier = () => {
|
||||
}
|
||||
|
||||
// Default fallback
|
||||
return 'app.timesafari';
|
||||
return 'app.timesafari.app';
|
||||
} catch (error) {
|
||||
console.error('Error getting app identifier:', error);
|
||||
return 'app.timesafari'; // Default fallback
|
||||
return 'app.timesafari.app'; // Default fallback
|
||||
}
|
||||
};
|
||||
|
||||
|
||||
Reference in New Issue
Block a user