Update README.md #124
Update README.md #124
Conversation
There was a problem hiding this comment.
@conrad-mal thanks so much for taking the initiative and contributing to our Ad Reporting package!! I have a few change requests in the below comments before this is ready for approval and to be merged into main.
Let me know if you have any questions when making the changes requested below. Thanks 😄
|
@conrad-mal I wanted to bump my change requests in this PR. Our team is planning to merge the next suite of changes to this repo in the upcoming PR #127. We would like to incorporate these changes as well; however, the above requested changes must be applied before we can do so. If you're able to make these changes in the next day or so that would be great! If not, our team will make these changes directly to your branch so they may be folded into the next release. Thanks and let me know if you have any questions! |
* release/v0.11.0 * Update CHANGELOG.md * Feature/linkedin click uri updates (#120) * feature/linkedin-click-uri-updates * docs regen and integration schema change * changelog update * integration test merge conflict address * changelog fix * another changelog fix * schema change * Apply suggestions from code review * Update README.md (#124) * Update README.md :) * Apply suggestions from code review --------- Co-authored-by: Joe Markiewicz <74217849+fivetran-joemarkiewicz@users.noreply.github.com> * contributor changelog update * Update CHANGELOG.md * regen docs --------- Co-authored-by: Joe Markiewicz <74217849+fivetran-joemarkiewicz@users.noreply.github.com> Co-authored-by: conrad-mal <conraddick@gmail.com>
:)