Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Download links #7132

Merged
merged 2 commits into from
Jan 12, 2022
Merged

Fix Download links #7132

merged 2 commits into from
Jan 12, 2022

Conversation

parasharrajat
Copy link
Member

Details

Fixed Issues

$ #7123

Tests | QA Steps

  1. Open app on Web
  2. Navigate to Settings > About > App download links > MacOS
  3. Install the app
  • Verify that no errors appear in the JS console

Tested On

  • Web
  • Mobile Web
  • Desktop
  • iOS
  • Android

Screenshots

Web

Mobile Web

Desktop

iOS

Android

I feel screenshots are not needed.

@parasharrajat parasharrajat requested a review from a team as a code owner January 11, 2022 19:49
@MelvinBot MelvinBot requested review from tgolen and removed request for a team January 11, 2022 19:49
@@ -1,5 +1,8 @@
import lodashGet from 'lodash/get';
import Config from 'react-native-config';
Copy link
Member Author

@parasharrajat parasharrajat Jan 11, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can't import the Config from ./CONFIG directly, Its undefined may be due to circular dependency.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried resolving it via creating another APP_URL file but still that was returning undefined.

@tgolen
Copy link
Contributor

tgolen commented Jan 12, 2022

Those constant names are getting confusing now. We might want to go back at some point and clean them up. Specifically the difference between NEW_EXPENSIFY_URL and CASH_URL.

@tgolen tgolen merged commit 2257a5b into Expensify:main Jan 12, 2022
@OSBotify
Copy link
Contributor

🚀 Deployed to staging by @tgolen in version: 1.1.27-7 🚀

platform result
🤖 android 🤖 failure ❌
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 failure ❌
🕸 web 🕸 success ✅

@OSBotify
Copy link
Contributor

🚀 Deployed to production by @Jag96 in version: 1.1.29-5 🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants