Table of Contents

Open questions

Business problem

This feature aims to implement the option to search for customers by phone in RBI Admin tool. This was a business request from Iberia (

Tech Discovery

Architecture AS-IS

Today, Admin Tool only has the option to search customers by email or Cognito ID. The search by email is exemplified by the following sequence diagram.

The most direct solution is using intl-packages, which already exposes a method to search users by phone number and DynamoDB queries for userLookupByPhone records associating phone numbers to Cognito IDs. However, there is a divergence between the records created using intl-packages and the ones created by user-service, as can be seen below:

intl-packages:

{
  pk: phoneNumber,
  pk2: cognitoId,
  sk: current/user-lookup-by-phone
  sk2: current/user-lookup-by-phone/{date} 
}

user-service:

{
  pk: phone#${phoneNumber},
  pk2: phone_user#${cognitoId},
  sk: v0_UserPhone,
  sk2: v0_UserPhoneUserPhoneLookUp,
}

The user-service record is created when a user verifies their phone number via SMS. As far as we can tell, no one is using intl-packages lookup records since its only reference is its own test.

This poses a problem, given that Iberia doesn’t require the customer to verify its phone, since they can use email.

Proposed solution

Add the option in intl-packagesto search users by phone number, and change user-service API to create/update lookupByPhoneNumber records whenever there are changes in the customer's phone number.

It’s important to notice that since lookupByPhoneNumber records haven’t been created for existing users, a script will have to be developed to scan all users in DynamoDB and create their associated lookup records.

Necessary changes - summary

• User Service API: Adapt GET user endpoint to also search by phone number

• User Service API: Adapt the create method in user service to create a lookupByPhoneNumber record if a phone number is informed

• User Service API: Adapt update-user lambda to update or create a lookupByPhoneNumber record if the phone number changed

• User Service API: Add a isPhoneNumberVerified field to lookupByPhoneNumber interface. This is important to avoid impacting https://rbictg.atlassian.net/wiki/spaces/EGMT/pages/4329078785/Solution+Phone+Number+Authentication+for+BK+SA#Customer-support-tool

• User packages: Change getByPhoneNumber to use User Service API instead of directly accessing the database

• Admin App Backend: Create a GraphQL query to request customers by phone number

• Admin App Frontend: Add the option to search customers by phone using the new query

• DynamoDB: Develop a migration script to create lookupByPhoneNumber for existing customers

Diagrams

Search customer by phone sequence diagram:

Necessary changes - details

(WIP)

intl-

packagesCreate the option to search lookupByPhone records created by

user-service

Expose the new searchByPhone option

This changes are already implemented in this git branch

1. Include the isVerified field in IUserLookupByPhone Interface: https://github.com/rbilabs/intl

-packages/pull/961

1. -user-service/blob/master/src/users/user-lookup-by-phone.repository.ts#L25-L30

Code Block
export interface IUserLookupByPhone { pk: string; pk2: string; sk: string; sk2: string; isVerified: boolean; }

2. Create lookupByPhoneNumber record after creating the user in dynamo inside the pre-signup lambda, but only if a phone number is informed: https://github.com/rbilabs/intl-user-service

In this service, the lookup record is created by createPhoneRecord function, which is called only in the verifyPhoneNumber method (

2. /blob/master/src/cognito-lambdas/pre-signup.service.ts#L135

3. Update the lookupByPhoneNumber record if an user update is triggered and includes the phone number: https://github.com/rbilabs/intl-user-service/blob/master/src/

verification

2. users/

verify-phone.service.ts#L112-L117)

There are a couple of possible solutions to create these records whenever a customer phoneNumber is created or updated, for example:

Call createPhoneRecord in both signUp user and update user details endpoints

Create a lambda responsible for creating and updating lookup records which is triggered using DynamoDB streams.

2. update-comm-pref.service.ts#L150

3. Adapt the get user endpoint to have an phoneNumber query field, that searches by phone: https://github.com/rbilabs/intl-user-service/blob/master/src/users/users.controller.ts#L124

intl-packages (WIP)

• Create the option to search lookupByPhone records created by user-service

• Expose the new searchByPhone option

This changes are already implemented in this git branch: https://github.com/rbilabs/intl-packages/pull/961

intl-admin-app

• Create a GraphQL query that uses the searchByPhone method exposed by intl-packages

• Update the UI to display the search by phone number option

These main changes are implemented in this git branch: https://github.com/rbilabs/intl-admin-app/pull/242

However, Admin App currently doesn’t use Launch Darkly, which makes it harder to control the feature activation for different markets and makes it riskier. Possible alternatives are:

	Pros	Cons
Add and configure Launch Darkly in Admin App	Easy and quick FT management Adhere to the pattern used in many other RBI repositories	High effort
Implement FT using environment variables	Medium effort	FT management requires that a Developer updates the CircleCI configuration and redeploys the app An excessive number of env vars might be necessary
Only display the search by phone option in Iberia market	Low effort	Activation in other markets will require code changes Once activated in an environment, can only be deactivated via hotfix or waiting for the normal Lifecycle