Skip to content
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.

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

SYNC_FIELDS Support #53

Open
jasondpq opened this issue Apr 28, 2016 · 3 comments
Open

SYNC_FIELDS Support #53

jasondpq opened this issue Apr 28, 2016 · 3 comments

Comments

@jasondpq
Copy link

jasondpq commented Apr 28, 2016

HI,

i am digging into the silverpop api and have just learned about the update_if_found and sync_fields flag of the addrecipient command.

is this supported? they are telling us that using a non keyed database is preferable when combing sms and email into a single DB. In order to update an existing record I went the easy route and set the UPDATE_IF_FOUND flag to true. one of the dbs i was writing to had no unique key set up and i got an error saying i needed to use SYNC_FIELDS on databases with no unique key if the UPDATE_IF_FOUND flag is set.

i don't see support for that here https://github.com/Silverpop/engage-api-client/blob/master/src/main/java/com/silverpop/api/client/command/AddRecipientCommand.java

our support rep sent over the following
`

85628 2 EMAIL [email protected] Customer Id 123-45-6789 Email [email protected] ``
@jasondpq
Copy link
Author

Does anyone have any ideas on this? i am starting to look at how i can implement and could use a little guidance :)

@smitchelus
Copy link
Contributor

Hi Jason, I don't have anyone available right now to add this, but we would certainly accept a pull request if you wanted to give it a try. It really wouldn't be complicated. You'd need a new SyncField class in the elements package modeled after the existing Column class, and then a new sync fields list instance variable in the AddRecipientCommand class using that new SyncField class. Then just setter/getter methods and I think that should do the trick.

@jasondpq
Copy link
Author

ok. I will see if i can put some time to this.

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

No branches or pull requests

2 participants