Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Webhook & schema changes to monitor custom contact info #71

Closed
4 tasks done
Satendra-SR opened this issue Mar 31, 2021 · 9 comments
Closed
4 tasks done

Webhook & schema changes to monitor custom contact info #71

Satendra-SR opened this issue Mar 31, 2021 · 9 comments
Assignees
Labels
enhancement New feature or request priority: medium This issue or pull request has medium priority status: ready to test

Comments

@Satendra-SR
Copy link
Member

Satendra-SR commented Mar 31, 2021

Is your feature request related to a problem? Please describe.
Initially, there was not much custom contact info that we were planing to capture, but as of now, we are capturing different custom contact information for a contact.
We need to capture these information in our database for generating relative matrices.

Describe the solution you'd like

  • List down the contact info we would like to capture.
  • Check whether we are getting these information from RP (We are getting contact group info, these should be available as well).
  • Come up with the new schema design based on the
  • Make code changes to implement the feature request.

Describe alternatives you've considered
None

Additional context
None

@Satendra-SR Satendra-SR added the enhancement New feature or request label Mar 31, 2021
@Satendra-SR Satendra-SR self-assigned this Mar 31, 2021
@Satendra-SR
Copy link
Member Author

@prtkdost Assigning it to you for first round. Can you take the first point. We will need a list of what all custom dataset we are capturing and that needs to be captured in out database as well.

@Satendra-SR Satendra-SR assigned prtkdost and unassigned Satendra-SR Mar 31, 2021
@Satendra-SR Satendra-SR added the priority: medium This issue or pull request has medium priority label Mar 31, 2021
@Satendra-SR
Copy link
Member Author

List down the contact info we would like to capture.

  • These are the dynamic contact fields and can be added more in future.
  • We need to come up with a plan so that we can incorporate new fields as well.

@Satendra-SR
Copy link
Member Author

Check whether we are getting these information from RP

We can get these contact fields details using contact.groups variables we have in RP.


RP flow post payload

image


Json object

image

@Satendra-SR
Copy link
Member Author

Satendra-SR commented Apr 15, 2021

Sharing the proposed schema for capturing custom contact details

Table name: user_contact

column type foreign key
id int
user_id int user -> id
registration_id int registration -> id
user_phone varchar
flow_run_uuid varchar
field_name varchar
field_value varchar
created_on timestamp
updated_on timestamp

@Satendra-SR
Copy link
Member Author

Capture only fields with prefix dost_

Satendra-SR added a commit that referenced this issue Apr 20, 2021
…or-custom-contact-info

#71 changes to monitor custom contact info
@Satendra-SR
Copy link
Member Author

This is deployed on UAT and ready for testing.

Note: Please use these two flows for testing or create new for testing.

  1. DOST-Test-P2M7D4-devteam
  2. FINAL-DOST-test-OnboardingCall-devteam

Steps to test:

  1. Schedule a call from above flows.
  2. Check user_custom_fields table for latest entry for your phone number.
  3. Check the table is populated with correct user contact information.

@Satendra-SR
Copy link
Member Author

This is deployed and working fine on production as well.

@Satendra-SR
Copy link
Member Author

Need to check dost_flow_status

@Satendra-SR
Copy link
Member Author

Working as expected on prod. Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request priority: medium This issue or pull request has medium priority status: ready to test
Projects
None yet
Development

No branches or pull requests

2 participants