Merge pull request #15726 from Expensify/cmartins-fixTestBuildLinks

[No QA] Fix test build links for Web/Desktop
Expensify · Mar 7, 2023 · 7e8e8a1 · 7e8e8a1
2 parents 5cfbea4 + 83e85f3
commit 7e8e8a1
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/.github/workflows/testBuild.yml b/.github/workflows/testBuild.yml
@@ -290,6 +290,6 @@ jobs:
           IOS: ${{ needs.iOS.result }}
           WEB: ${{ needs.web.result }}
           ANDROID_LINK: ${{fromJson(steps.get_android_path.outputs.android_paths).html_path}}
-          DESKTOP_LINK: "https://ad-hoc-expensify-cash.s3.amazonaws.com/desktop/$PULL_REQUEST_NUMBER/NewExpensify.dmg"
+          DESKTOP_LINK: https://ad-hoc-expensify-cash.s3.amazonaws.com/desktop/${{ env.PULL_REQUEST_NUMBER }}/NewExpensify.dmg
           IOS_LINK: ${{ fromJson(steps.get_ios_path.outputs.ios_paths).html_path }}
-          WEB_LINK: "https://$PULL_REQUEST_NUMBER.pr-testing.expensify.com"
+          WEB_LINK: https://${{ env.PULL_REQUEST_NUMBER }}.pr-testing.expensify.com