wallet-selector icon indicating copy to clipboard operation
wallet-selector copied to clipboard

feat: finer-wallet support

Open prih opened this issue 2 years ago • 15 comments

Description

Add FiNER Wallet App support.

FiNER Wallet implements an API almost identical to near-wallet over Deep linking on IOS and Android.

Checklist:

  • [x] I have performed a self-review of my own code
  • [x] I have commented my code, particularly in hard-to-understand areas
  • [x] I have made corresponding changes to the documentation
  • [x] My changes generate no new warnings

Type of change. This type of change is the main reason for the PR.

  • [x] FEATURE - a PR of this type introduces a new feature.

Breaking changes

  • [x] NO BREAKING CHANGE - this PR doesn't contain any breaking changes and it's backwards compatible

prih avatar Aug 02 '22 22:08 prih

Hey @prih.

I cloned your repo and checked out the fintoda:dev branch. I wanted to test Finer wallet before I can approve this PR. When I tried to connect with Finer Wallet it was not working. I got this error:

Failed to launch 'finer://wallet.testnet.near.org/login/?success_url=http%3A%2F%2Flocalhost%3A4200%2Fwallet-selector&failure_url=http%3A%2F%2Flocalhost%3A4200%2Fwallet-selector&contract_id=guest-book.testnet&public_key=ed25519%3A4Tt9pYDsFpnhTeDEs8Ch4PioWFfLc5he4qKZ7n2EJtsP' because the scheme does not have a registered handler.

My questions:

  1. Is there a link to more information about Finer Wallet? I only found this link: https://finerwallet.io/.
  2. Can you help me understand how I can connect to the Finer Wallet in your branch so I can test?

amirsaran3 avatar Sep 08 '22 08:09 amirsaran3

@amirsaran3 hi,

finer:// scheme is available on the mobile platform, Finer is a mobile application. On a mobile operating system, an application can assign a schema to itself.

prih avatar Sep 08 '22 09:09 prih

@amirsaran3 hi,

finer:// scheme is available on the mobile platform, Finer is a mobile application. On a mobile operating system, an application can assign a schema to itself.

Thanks for the reply @prih. I can't find the finer wallet in the Play Store.

How can I test the implementation of the finer wallet while developing?

amirsaran3 avatar Sep 08 '22 10:09 amirsaran3

@amirsaran3 hi, finer:// scheme is available on the mobile platform, Finer is a mobile application. On a mobile operating system, an application can assign a schema to itself.

Thanks for the reply @prih. I can't find the finer wallet in the Play Store.

How can I test the implementation of the finer wallet while developing?

Finer is only available on iOS App Store (we are currently working on delivery to the Google Play Market)

prih avatar Sep 08 '22 11:09 prih

Hey @prih. I checked out your latest changes and it is not quite the intended behavior. There is no "Not Installed" flow, it immediately redirects to your website.

Have you tried using deferred deep linking to redirect to app store if app is not installed?

amirsaran3 avatar Sep 12 '22 11:09 amirsaran3

Hey @prih. I checked out your latest changes and it is not quite the intended behavior. There is no "Not Installed" flow, it immediately redirects to your website.

Have you tried using deferred deep linking to redirect to app store if app is not installed?

We have no way to track if the app is installed or not, it's not a desktop web and it's not a browser extension. We have implemented support for communication through Universal Links (https://developer.apple.com/ios/universal-links/), the user gets to an intermediate page on the site, if he has the application installed, then the browser will offer to open the current url in the Finer application, also, if the browser is different from Safari, the page has a code that tries to follow the current link by changing the current scheme from https:// to finer://, and if the user has the application installed, then a transition will occur. In other cases, the user remains on the page where the link to go to the App Store is located. If you have a better idea, please suggest)

prih avatar Sep 12 '22 15:09 prih

Hey @prih as a reminder, the security statement needs to be shown either on your GitHub repo or the website. You can check it here again. Thanks!

AmmarHumackicSQA avatar Sep 16 '22 12:09 AmmarHumackicSQA

Hey @prih as a reminder, the security statement needs to be shown either on your GitHub repo or the website. You can check it here again. Thanks!

Hi @prih can we get a confirmation on this? Thanks.

AmmarHumackicSQA avatar Sep 20 '22 17:09 AmmarHumackicSQA

yes, confirm

prih avatar Sep 20 '22 17:09 prih

yes, confirm

Can we get a link to it? We couldn't find it on your website. The security statement needs to be shown either on your GitHub repo or the website (explained here)

AmmarHumackicSQA avatar Sep 20 '22 17:09 AmmarHumackicSQA

yes, confirm

Can we get a link to it? We couldn't find it on your website. The security statement needs to be shown either on your GitHub repo or the website (explained here)

we are actively developing the site and the support system, I will soon be able to provide. Main domain is https://finerwallet.io

prih avatar Sep 20 '22 17:09 prih

@prih There are conflicts right now. Can you resolve them when you have time and we can merge this PR.

amirsaran3 avatar Sep 23 '22 08:09 amirsaran3

yes, confirm

Can we get a link to it? We couldn't find it on your website. The security statement needs to be shown either on your GitHub repo or the website (explained here)

we are actively developing the site and the support system, I will soon be able to provide. Main domain is https://finerwallet.io

Hi @prih any news on this? Thanks.

AmmarHumackicSQA avatar Sep 29 '22 11:09 AmmarHumackicSQA

yes, confirm

Can we get a link to it? We couldn't find it on your website. The security statement needs to be shown either on your GitHub repo or the website (explained here)

we are actively developing the site and the support system, I will soon be able to provide. Main domain is https://finerwallet.io

@prih pinging to check the status of this, and to see if this PR could be potentially included in the next release of Wallet Selector. Thanks!

AmmarHumackicSQA avatar Oct 11 '22 13:10 AmmarHumackicSQA

we are working on resolving this issue

prih avatar Oct 11 '22 14:10 prih

we are working on resolving this issue

Hey @prih any news on the status regarding this? Thanks!

AmmarHumackicSQA avatar Nov 04 '22 15:11 AmmarHumackicSQA

@prih gentle bump 🙂

AmmarHumackicSQA avatar Nov 28 '22 07:11 AmmarHumackicSQA