simplify logic for pull request OTAs

.github/workflows/pull-request-comment.yml

@@ -171,8 +171,6 @@ jobs:
             Your requested OTA deployment was successful! You may now apply it by pressing the link below.
 
             [Apply OTA update](bluesky://intent/apply-ota?channel=pull-request-${{ github.event.issue.number }})
-
-            [Here is some music to listen to while you wait...](https://www.youtube.com/watch?v=VBlFHuCzPgY)
             ---
             *Generated by [PR labeler](https://github.com/expo/expo/actions/workflows/pr-labeler.yml) 🤖*
 

app.config.js

@@ -26,8 +26,7 @@ module.exports = function (_config) {
     ...(IS_DEV || IS_TESTFLIGHT ? [] : []),
   ]
 
-  const UPDATES_ENABLED =
+  const UPDATES_ENABLED = IS_TESTFLIGHT || IS_PRODUCTION
-    IS_TESTFLIGHT !== undefined || IS_PRODUCTION !== undefined
 
   const USE_SENTRY = Boolean(process.env.SENTRY_AUTH_TOKEN)
 

src/lib/hooks/useOTAUpdates.ts

@@ -45,7 +45,6 @@ async function updateTestflight() {
   const res = await checkForUpdateAsync()
   if (res.isAvailable) {
     await fetchUpdateAsync()
-
     Alert.alert(
       'Update Available',
       'A new version of the app is available. Relaunch now?',
@@ -70,43 +69,13 @@ export function useApplyPullRequestOTAUpdate() {
   const {currentlyRunning} = useUpdates()
   const [pending, setPending] = React.useState(false)
   const currentChannel = currentlyRunning?.channel
-  const isCurrentlyRunningPullRequestDeployment =
-    currentChannel?.startsWith('pull-request')
 
   const tryApplyUpdate = async (channel: string) => {
     setPending(true)
-    if (currentChannel === channel) {
+    await setExtraParamsPullRequest(channel)
     const res = await checkForUpdateAsync()
     if (res.isAvailable) {
-        logger.debug('Attempting to fetch update...')
       await fetchUpdateAsync()
-        Alert.alert(
-          'Deployment Available',
-          `A new deployment of ${channel} is availalble. Relaunch now?`,
-          [
-            {
-              text: 'No',
-              style: 'cancel',
-            },
-            {
-              text: 'Relaunch',
-              style: 'default',
-              onPress: async () => {
-                await reloadAsync()
-              },
-            },
-          ],
-        )
-      } else {
-        Alert.alert(
-          'No Deployment Available',
-          `No new deployments of ${channel} are currently available for your current native build.`,
-        )
-      }
-    } else {
-      setExtraParamsPullRequest(channel)
-      const res = await checkForUpdateAsync()
-      if (res.isAvailable) {
       Alert.alert(
         'Deployment Available',
         `A deployment of ${channel} is availalble. Applying this deployment may result in a bricked installation, in which case you will need to reinstall the app and may lose local data. Are you sure you want to proceed?`,
@@ -130,7 +99,6 @@ export function useApplyPullRequestOTAUpdate() {
         `No new deployments of ${channel} are currently available for your current native build.`,
       )
     }
-    }
     setPending(false)
   }
 
@@ -146,7 +114,6 @@ export function useApplyPullRequestOTAUpdate() {
     tryApplyUpdate,
     revertToEmbedded,
     currentChannel,
-    isCurrentlyRunningPullRequestDeployment,
     pending,
   }
 }