intercom / intercom-dotnet

Intercom API client library for .NET
https://developers.intercom.io/reference
Apache License 2.0
63 stars 53 forks source link

DotNetStandard 2.0 Support - roadmap #90

Closed kmossco closed 6 years ago

kmossco commented 6 years ago

Thanks to the continued feedback from the community, we have now prioritised in our roadmap to implement these bindings in DotNetStandard 2.0. We haven't yet decided if we will keep RestSharp as a dependency but we hope to have taken a decision and started implementation in the next couple of weeks.

And in order to keep this discussion more organised and cleared, I have now closed the other issues so we can continue discussing this here.

For the sake of transparency this is our plan:

Thank you so much for everyone that submitted a PR or showed us a fork with an update on this, we really appreciate all your efforts! I'm sure we will have an update soon, and feel free to drop us a comment here if you have any concerns or questions. 👍

nicholi commented 6 years ago

As I stated in https://github.com/intercom/intercom-dotnet/issues/43, there should be zero modifications necessary for the Library code when updating to the latest RestSharp netstandard release (which is now stable).

You can see my branch here: https://github.com/Nicholi/intercom-dotnet/tree/netstandard20 The only things you have to do is just create new csproj files really. The new netcore csproj files are VERY minimal, requires like zero effort.

The only real changes I made were for the Test projects, so that the nunit tests would also be netcore compatible (which meant upgrading to nunit 3.x), and there were a few deprecated things that had to be changed. All very minor and simple in and of themselves though.

All tests are still passing and I've had no issue using the library in netcore projects yet.

kmossco commented 6 years ago

I have now pushed a PR that brings all this discussion into a conclusion. I also left a few notes on my train of thought in the PR comment. I'll close this when the PR is merged, but if anyone reading this could test that branch and see if it works for them it would be greatly appreciated! 🎉

neooleg commented 6 years ago

@kmossco could you pls update -- is there is any progress?

kmossco commented 6 years ago

So sorry for the delay in getting this out of the door. We unfortunately hit a snag creating the Nuget package in a way that it doesn't break plenty of apps that don't want to upgrade the library, and we are working to create a new release. We should have an update very soon though!

neooleg commented 6 years ago

Thanks for update, Bruno.

Looking forward for Core release!

On Sat, Mar 17, 2018, 22:37 Bruno Vieira notifications@github.com wrote:

So sorry for the delay in getting this out of the door. We unfortunately hit a snag creating the Nuget package in a way that it doesn't break plenty of apps that don't want to upgrade the library, and we are working to create a new release. We should have an update very soon though!

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/intercom/intercom-dotnet/issues/90#issuecomment-373950818, or mute the thread https://github.com/notifications/unsubscribe-auth/ABkhyQcxi3i0D9aYC4s6CxMA4eRVoD8uks5tfXQSgaJpZM4Qp3o3 .

-- Best regards, Oleg

NPSF3000 commented 6 years ago

@kmossco thanks for the update 😊

niemyjski commented 6 years ago

+1, also can we get a fully async api :D

kmossco commented 6 years ago

@niemyjski that's definitely in our plans too! 😄

kmossco commented 6 years ago

Closing this as we have now a new version out that supports DotNetStandard. Going through the issues already open and will work next on adding the missing features to the SDK. Please let us know if you spot any problems/errors. 🎉