Refactor - separate DDL event handlers#114
Refactor - separate DDL event handlers#114recurlybrad wants to merge 1 commit intodata-integrations:developfrom
Conversation
|
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed (or fixed any issues), please reply here with What to do if you already signed the CLAIndividual signers
Corporate signers
ℹ️ Googlers: Go here for more info. |
f46fd21 to
d3d55a0
Compare
|
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed (or fixed any issues), please reply here with What to do if you already signed the CLAIndividual signers
Corporate signers
ℹ️ Googlers: Go here for more info. |
This refactor could use slightly more work to encapsulate just the bits needed by each DDL Event Handler a little more.
Overall gist of this is to make the code a bit more manageable and possibly allow for dependency injected Event Handlers.