question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

[IOU] Add isAppInstalled logic to Expensify.cash

See original GitHub issue

If you haven’t already, check out our contributing guidelines for onboarding!


view the post on Upwork here.

Platform - version:
current

We require a method of detecting installed apps for the Expensify.cash app on Android & iOS.

  1. Install and implement the react-native-check-app-install package.
  2. Create a helper function under libs isAppInstalled() (similar to getPlatform) which takes two params: iosScheme, androidPackageId, and returns true if the selected app is installed.
  3. Within the helper function, getPlatform() should be used to determine which library function to use:
    • Android: AppInstalledChecker.isAppInstalledAndroid(androidPackageId)
    • iOS: AppInstalledChecker.checkURLScheme(iosScheme)
  4. As this is only used on mobile, this should be build within an index.native.js file (unfortunately the params are different for iOS/Android, so we cant create index.android.js|index.ios.js``). Also, the default index.js function should always return false.

Issue Analytics

  • State:closed
  • Created 3 years ago
  • Comments:9 (8 by maintainers)

github_iconTop GitHub Comments

1reaction
SameeraMadushancommented, Feb 23, 2021

Thanks for all of your proposals and comments! @Maftalion I’m going to pass this issue onto the second responder as you were hired for #1508.

@SameeraMadushan, would you like to move forward with the implementation? Thanks for your suggestion about reusing checkURLScheme for Android, we should also be able to use a single native.js file for both Android & iOS.

Thank you @Julesssss I accept the offer. We can handle it inside native.js. I will work on this 👍🏻

1reaction
Julessssscommented, Feb 23, 2021

Thanks for all of your proposals and comments! @Maftalion I’m going to pass this issue onto the second responder as you were hired for #1508.

@SameeraMadushan, would you like to move forward with the implementation? Thanks for your suggestion about reusing checkURLScheme for Android, we should also be able to use a single native.js file for both Android & iOS.

Read more comments on GitHub >

github_iconTop Results From Across the Web

[IOU] Add isAppInstalled logic to Expensify.cash #1526 - GitHub
Create a helper function under libs isAppInstalled() (similar to getPlatform) which takes two params: iosScheme, androidPackageId , and returns true if the ...
Read more >
Adding Payment Methods - Expensify Community
If you are tracking your expenses that are reimbursable, they default to cash. It won't get granular and say cash or type of...
Read more >
Adding more export options for your reports
Today we're happy to announce a handful of new default exports available in every Expensify account to make this process even easier. While...
Read more >
Expensify Terms of Service | Our Terms of Use
To the extent permitted by law, you grant Expensify the right to apply or set off any money or credit balance in which...
Read more >
Expensify: Easy Money
Sign up or log in to automate your preaccounting process for expenses, bills, invoices, and more! Start a free trial and see why...
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found