Skip to content
This repository has been archived by the owner on Aug 29, 2024. It is now read-only.

feat(FTL-7654): added update to SDK to support the Truecaller flow #267

Merged
merged 11 commits into from
Nov 28, 2022

Conversation

oleksiipiliugin
Copy link
Collaborator

@oleksiipiliugin oleksiipiliugin commented Nov 14, 2022

Implementation of the signin flow with the Truecaller profile/token.

Logic is the same as for sigin passwordless with phone number, but without sending OTP to the user.
With this new feature, we are able to create a new user in the Cognito pool or login if it exists.
The main changes are in sdk/core and lib/user-management. Also, support for new endpoints was added for the internal key storage service.

@oleksiipiliugin oleksiipiliugin changed the title feat(FTL-7654): added update SDK to support the Truecaller flow feat(FTL-7654): added update to SDK to support the Truecaller flow Nov 14, 2022
Copy link
Contributor

@yaroslava-kurash yaroslava-kurash left a comment

Choose a reason for hiding this comment

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

Shouldn't versions of @affinidi/wallet-browser-sdk @affinidi/wallet-expo-sdk @affinidi/wallet-node-sdk and @affinidi/wallet-react-native-sdk be updated too as they use new version of @affinidi/wallet-core-sdk

require('./services')
require('./AffinityWallet.test')
require('./NetworkMember.test')
// require('./services')
Copy link
Contributor

@linn-affinidi linn-affinidi Nov 28, 2022

Choose a reason for hiding this comment

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

to enable back?

@@ -643,3 +658,79 @@ parallel('CommonNetworkMember [OTP]', () => {
})
})
})

// describe('CommonNetworkMember [TrueCaller]', () => {
Copy link
Contributor

Choose a reason for hiding this comment

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

@oleksiipiliugin why we comment it out ?

Copy link
Contributor

Choose a reason for hiding this comment

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

ah we probably need to deploy awb first

@oleksiipiliugin oleksiipiliugin merged commit c5e739b into master Nov 28, 2022
@oleksiipiliugin oleksiipiliugin deleted the feat/FTL-7741 branch November 28, 2022 16:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants