react-native-code-push icon indicating copy to clipboard operation
react-native-code-push copied to clipboard

Move DevSettingsActivity from main to debug

Open floydkim opened this issue 2 years ago • 1 comments

closes: https://github.com/microsoft/react-native-code-push/issues/2353

Tests

  • [x] npm run test:android
  • [x] building app, installing code-push bundle
    • on production Android app (RN 0.67.5, with same modification on [email protected] package using patch-package)
  • [x] DevSettingsActivity not found in manifest-merger-release-report.txt

For anyone looking for a patch for this issue, here it is.
diff --git a/node_modules/react-native-code-push/android/app/src/debug/AndroidManifest.xml b/node_modules/react-native-code-push/android/app/src/debug/AndroidManifest.xml
new file mode 100644
index 0000000..5bd1596
--- /dev/null
+++ b/node_modules/react-native-code-push/android/app/src/debug/AndroidManifest.xml
@@ -0,0 +1,10 @@
+<manifest xmlns:android="http://schemas.android.com/apk/res/android"
+    package="com.microsoft.codepush.react">
+
+    <uses-permission android:name="android.permission.INTERNET" />
+
+    <application>
+        <activity android:name="com.facebook.react.devsupport.DevSettingsActivity" />
+    </application>
+
+</manifest>
\ No newline at end of file
diff --git a/node_modules/react-native-code-push/android/app/src/main/AndroidManifest.xml b/node_modules/react-native-code-push/android/app/src/main/AndroidManifest.xml
index 5bd1596..9b2755f 100644
--- a/node_modules/react-native-code-push/android/app/src/main/AndroidManifest.xml
+++ b/node_modules/react-native-code-push/android/app/src/main/AndroidManifest.xml
@@ -3,8 +3,4 @@
 
     <uses-permission android:name="android.permission.INTERNET" />
 
-    <application>
-        <activity android:name="com.facebook.react.devsupport.DevSettingsActivity" />
-    </application>
-
 </manifest>

floydkim avatar Aug 12 '23 10:08 floydkim

@microsoft-github-policy-service agree

floydkim avatar Aug 19 '23 10:08 floydkim

Hi @floydkim, merging this PR would mean you will be updating the react-native-code-push package to latest version once it is released. I would recommend trying to update it to the current latest version 8.2.2 and test if it works with your [email protected] project. Since version 0.67 is archived, I also recommend migrating to a newer React Native version if you haven't done so yet.

DmitriyKirakosyan avatar Jun 04 '24 08:06 DmitriyKirakosyan

hello! @DmitriyKirakosyan It's been about 10 months since I created this PR and the app I'm working on is now using react-native 0.73 with [email protected] based package. If this PR is merged, I'd be happy to test the new release on the app. Thank you.

floydkim avatar Jun 04 '24 16:06 floydkim

Also related https://github.com/microsoft/react-native-code-push/issues/2658/

DmitriyKirakosyan avatar Jun 11 '24 05:06 DmitriyKirakosyan