Importing "Active On Site" and other metric-types

[jobondi]

This tap has been very helpful for us to quickly automate ETL and get detailed analytics going in our data warehouse for email metrics (receives, opens, clicks, bounces, etc) and we'd like to also pull in metrics that Klaviyo tracks for user retention and site-activity metrics (Active On Site).

Has anyone thought of or tried expanding this tap to optionally pull in more metric-types, either as separate optional tables (one for each metric type) or one table with all the non-email-related metrics?

If not, we be happy to take a shot at adding this functionality to this tap. What's the preferred procedure for this, create a fork or a branch?

[gmfennema]

Hey @jobondi I solved this one:
https://github.com/gmfennema/tap-klaviyo/commit/17726d4dbfc7d41487031a521f5ed7425346102e

[jobondi]

That's great @gmfennema! Is there any chance these changes could be submitted & proposed as a PR in this main repo? Mainly asking since if it gets merged to master then it should get pulled into Stitch Data and work automatically, not requiring setup of any scheduled ETL infrastructure + environment.