Skip to content

CDAP-15887. SendGrid Source #1

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 7 commits into
base: develop
Choose a base branch
from
Open

CDAP-15887. SendGrid Source #1

wants to merge 7 commits into from

Conversation

@dgrinenko
Copy link
Owner

@dgrinenko dgrinenko commented Oct 22, 2019

What is done:

  • Implemented Source Plugin
  • Implemented Multi Object Source Plugin

ToDo:

  • Customize widget filters and properties layout
  • Add possibility to pass optional arguments from configuration to Request
  • Add validation
  • Add integrationtests
  • Add sink

What is tested:

  • work on real SendGrid API with prepared data
  • work on cdap pipelines , transform data and pass them to some sinks

@dgrinenko dgrinenko self-assigned this Oct 22, 2019
@dgrinenko
General huge stuff
a1900eb 
- pom.xml
- object definitions
- icons
- widget
- *.xml
aonischuk
aonischuk suggested changes Oct 23, 2019
View reviewed changes
aonischuk
aonischuk suggested changes Oct 23, 2019
View reviewed changes
@dgrinenko dgrinenko force-pushed the CDAP-15887 branch 6 times, most recently from e61ff61 to a8ee900 Compare October 26, 2019 23:02
dgrinenko pushed a commit that referenced this pull request May 11, 2020
Merge pull request #1 from data-integrations/init
62eea60 
init sendgrid postaction
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@aonischuk aonischuk aonischuk requested changes

Reviewers whose approvals may not affect merge requirements

Assignees

@dgrinenko dgrinenko

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@dgrinenko @aonischuk @mikkin