Skip to content

Conversation

@tjennison-work
Copy link

We talked about putting this in a separate tool but I'll have to move a bunch of code into shared locations for that so have a look at this first since it's already done. I'll change it if you still want to go that route.

@kemp-google
Copy link

Sorry I missed this entirely.

I do think we want to split it out, I don't want the v1 service plumbed here. Looks like the common code is the pagination handling and BQ integration mostly?

@tjennison-work tjennison-work force-pushed the dev branch 3 times, most recently from d64f463 to a2a06b4 Compare November 20, 2018 20:04
@tjennison-work
Copy link
Author

I've split off the v1 export into a separate tool, which required moving the client setup and export code into shared packages.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants