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

Add user_id to CreateOrUpdateSubscriber #14

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mpnow
Copy link

@mpnow mpnow commented Aug 23, 2017

Add support for specifying the user_id field when creating or updating a subscriber.

I also updated the unit tests to support and test this field.

Note that this change was only made to the VS 2015 solution. I don't have access to VS 2013 to make the change in the VS 2013 solution, and my recommendation is that you remove that solution from the .NET SDK as it is now two versions old with the recent introduction of VS 2017.

@theDL
Copy link

theDL commented Jun 18, 2018

was this PR rejected for some reason, or was the repo simply abandoned?

@will-in-wi
Copy link
Contributor

It's abandoned at this point. We haven't put the resources into keeping it up to date and working. We've been discussing applying people to this internally.

I'll take a look at this PR and get you a better answer.

@KarlJorgen
Copy link

Hi @will-in-wi
Any update with this PR?

I hope you find the time internally to update this project in the future.
But can you please review this PR and merge it if it is ok?
We need user_id in our solution :)

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.

4 participants