Table of Contents
| Table of Contents |
|---|
Open questions
Business Questions
Is there a business rule that demands that only verified phone numbers can be used in customer search?
Business Answers
Tech Questions
Why do two different Interfaces for lookupByPhone records exist? (one in
user-serviceand another inintl-packages)Does accessing user-service phone lookup records from intl-packages violate the current architecture?
Since Admin App is not using Launch Darkly, does it have a strategy defined for Feature Toggles?
Tech Answers
This is legacy from US, and neither of them are used today, since sign up via phone is disabled in international.
Yes, we should create a new endpoint in user-service to search by phone number.
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 (
| Jira Legacy | ||||||
|---|---|---|---|---|---|---|
|
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:
| Code Block | ||
|---|---|---|
| ||
{
pk: phoneNumber,
pk2: cognitoId,
sk: current/user-lookup-by-phone
sk2: current/user-lookup-by-phone/{date}
} |
user-service:
| Code Block | ||
|---|---|---|
| ||
{
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.
Proposed solution n. 1
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.
This poses a problem, given that Iberia doesn’t require the customer to verify its phone, the number of users with a verified number is extremely low, since they can use email
Proposed solution n. 2
Based on the solution n.1 and the problem highlighted, a potential solution would be After that, it would be necessary to save the phone number added into the next delivery purchase (based on the user) as the validated number (when the phone does not exist).
| Info |
|---|
PROS: The operator of the Admin tool can search by phone. |
| Note |
|---|
CONS: There will be a phone number that is not officially validated and it can generate cases where the operator will input a phone number that is not the one expected from the user, making the search not valid. |
Rule on when and how to save the phone number need to be defined
Proposed solution n. 3 - deprecated
The same approach could be handled on the order side but, until now, we couldn’t find a way to connect the order with a user (no unique IDs existing).