SelfKeyFoundation / Identity-Wallet

Code for the SelfKey Identity Wallet
https://selfkeyfoundation.github.io/Identity-Wallet/
MIT License
57 stars 26 forks source link

Add Korporatio to Incorporations Marketplace #1771

Closed CortecKYC closed 3 years ago

CortecKYC commented 4 years ago

Once their KYCC instance is up and running in production, add Korporatio to Incorporations Marketplace as the exclusive vendor for Seychelles.

rhdelima commented 4 years ago

@sk91 @andregoncalves I don't see Seychelles on the Incorporation Marketplace. Is this an airtable issue? Build: dev_2019.12.19.11

andregoncalves commented 4 years ago

What env variables are you using?

rhdelima commented 4 years ago

@andregoncalves I'm using these:

  1. MATOMO_SITE=2 ATTRIBUTE_TYPE_SOURCE_OVERRIDE='development' FEATURE_CORPORATE_MARKETPLACE=1
  2. MATOMO_SITE=2 only Can you share what env variables to use?
andregoncalves commented 4 years ago

FORCE_DEV_MODE=1 should work. Korporatio is not active yet in prod airtable.

rhdelima commented 4 years ago

@andregoncalves Please see issues below:

  1. Error when submitting requirements. Env variables: MATOMO_SITE=2 FORCE_DEV_MODE=1 INCORPORATIONS_PRICE_OVERRIDE=0 PRIMARY_TOKEN=KI CHAIN_ID=3 open SelfKey\ Identity\ Wallet.app image image

  2. Missing link for terms and conditions image

CortecKYC commented 4 years ago

@andregoncalves ^^

andregoncalves commented 4 years ago

There's a PR for fixing the missing terms and conditions link. The issue with the submission is related to the test instance (not the same instance used in production). This doesn't block the release but it should be fixed, so until that happens this issue should not be closed.

andregoncalves commented 4 years ago

@rhdelima you can retest now, the KYCC test instance for Korporatio should be working now. Let me know if you need access credentials to the instance.

rhdelima commented 4 years ago

@andregoncalves I submitted an application but I don't see it on the korporatio test instance. MATOMO_SITE=2 FORCE_DEV_MODE=1 INCORPORATIONS_PRICE_OVERRIDE=0 PRIMARY_TOKEN=KI CHAIN_ID=3 DEV_TOOLS=yes open SelfKey\ Identity\ Wallet.app image image

andregoncalves commented 4 years ago

@rhdelima That's not the korporatio test instance. that's their production instance! *(Note: edited to delete Korporatio's test instance)

rhdelima commented 4 years ago

@andregoncalves my bad. Can you share the credentials, please?

andregoncalves commented 4 years ago

@rhdelima shared on mattermost.

rhdelima commented 4 years ago

@andregoncalves Please see my observations:

  1. I think passport and selfie should be separated in the template.
  2. Also, unable to view passport attachments. https://recordit.co/GoA69lW4i6
andregoncalves commented 4 years ago
  1. We have been discussing that change - but currently not possible
  2. This seems to be KYCC problem - @CortecKYC who should I ping on kycc team to investigate?
CortecKYC commented 4 years ago

@andregoncalves Yes, please

susananakatani commented 4 years ago

@andregoncalves Please see my observations:

  1. I think passport and selfie should be separated in the template.
  2. Also, unable to view passport attachments. https://recordit.co/GoA69lW4i6

@andregoncalves do you mean that KYCC instances integrated in the marketplace ( like Korporation) are not using the ID Verification tool for processes started from the marketplace?

CortecKYC commented 4 years ago

This ticket is on hold. KYCC devs are working on a temporary solution until v5 is rolled out (v5 would solve this issue but it's not ready yet).

CortecKYC commented 4 years ago

Followed up on Mattermost: https://mattermost.kyc-chain.com/kyc-chain/pl/t1ie55b16fb9ze5tehppcbs5ro

tl;dr KYCC v5 is not ready yet.

CortecKYC commented 4 years ago

Korporatio was added to the Marketplace. This ticket is still on QA because of the KYCC passport/selfie issue.