-
Notifications
You must be signed in to change notification settings - Fork 59
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
Need version for VS2017 #51
Comments
Hi, I've forked this repo and updated references and the vsixmanifest and now I can install and run in VS2017, but only for ODataV4. |
The beauty of open source, thanks @robsiera |
@robsiera I still get http://i.imgur.com/9Tn75KZ.png when I try to "Update service reference" in VS2017 if I compile your version, is there something else that needs to be changed for that to work? |
@robsiera Tried your release file, uninstalled my own and installed that one, but still gets the same error. Is there a log printed somewhere with a more detailed error? |
@robsiera Ah my bad, I thought I could reuse my existing Service reference and just do "update service reference" but if I added it as a Connected service instead it worked fine. Is there any major difference between the 2 types? |
no difference at all. |
@robsiera Okay, could hope for better git merge support :) we always gets merge conflicts if one person have added 1 property or class to the reference |
Thanks for the VS 2017 version but I get this error (https://snag.gy/tSZWjU.jpg) when I try update my old connected service or create a new one. Any idea to solve this? |
@aferreira44 I forked @robsiera code and fixed this issue. It was because the v3codegen had a dependency on the WCF Data Services been installed. Here is the blame |
@martinholden-skillsoft This is great! I don't know how to build this project but I do get the same error when I use the extension, trying to access data from the official OData sample server. I was hoping that if I ever got the SAP OData server we have on site working this would still happen. By the way, would you know how to fix this error of mine? #56. I don't know if it has something to do with an invalid ssl cert or not. I can't find where in the code the message is generated from. |
Addressed with PR #62! Please take a look at the Marketplace for the updated extension that should now support VS2017. Thank you for your support! |
Any plans for supporting VS 2017 ? I tried opening the Vsix and changing to version 15, but that didnt work
The text was updated successfully, but these errors were encountered: