search

Gary-Community-Ventures / benefits-api

MIT License
6 stars 5 forks source link

Optional Contact Storage Refactor #623

Open Linda-Vu opened 4 months ago

Linda-Vu commented 4 months ago

Refactor the current Hubspot code to include a base class definition which will allow us to implement Brevo as a solution for the NC while maintaining the Hubspot functionality in the Colorado instance.

Linda-Vu commented 4 months ago

@bespinosa01, @CalebPena could y'all see if this ticket is ok? feel free to add better acceptance criteria! tysm :)

CalebPena commented 4 months ago

Something like

class CrmIntegration:
    def __init__(self, user: User, screen: Screen):
         ...

    def add(self):
         ...

    def update(self):
        ...

    # if you want to have different criteria for integrating,
    # then we can both have a method to determine if a user should be uploaded to the CRM
    def should_integrate(self):
        ...
Linda-Vu commented 4 months ago

@bespinosa01 @mexi-cano we need to start on this in the new sprint (starting Monday) to enable the Brevo integration - any thoughts on who I should assign it to?