-
Notifications
You must be signed in to change notification settings - Fork 25
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
Type.ToTypeScriptType Generating Incorrectly #38
Comments
|
Something weird is going on, there are unit tests for those cases, and they are passing with flying colours. Could you prepare a simple minimal project that exposes problems and upload it on github? Also what version of VS are you using? |
Simple, minimal project works as expected. Using latest VS 2019 updates. I'll see if I can debug the plugin itself (not familiar with the process, but I'll take a look), to see what about this project would cause it. I m using EFCore 6, and these are models for that, but even after removing EF attributes they are rendering like that. |
OK, I did not figure out how to debug it, however I tried with the https://github.com/TietoEVRY/NTypewriterCLI CLI interface, and that worked correctly, when pointing at the project file. But that lead me to review the project in more detail, and I realized this is VS2019, but the project is itself the RC build of .NET6. If I change the TargetFramework to net5.0, it works. So this is on me for using RC build. But it may be worth identifying the cause, as it is RC now but it will be launching in a little over a month! I can try and dig a little more when/if I get some time, but feel free to close this if you don't want to resolve for .NET6 yet. |
How did you manage to build/compile .net 6.0 project with VS 2019? As far as I know, VS 2019 does not support .net 6.0 projects, you can open them, but you will not be able to build/compile... |
@vcastellano-spl congrats for diving into the problem and helping out the maintainer of the project, not something many people do! |
@NeVeSpl Long story, but this was because I was developing on two different machines, and the C# code was on a machine I was using VS2022 Preview on, and the frontend code was being developed on a different machine, and I was just trying to get the models to be available (but not compiling or running). Anyways, I am sorry for the confusion, I should have known it would be something like this. We can close this out as there's already a VS2022. @Etchelon I should move it over, I'm newer to TS, and used the example for the template in the comments #34, which uses commas (which seem to be supported since TS1.6?) but agreed best practices show as |
Installed from Visual Studio "Manage Extensions", version 0.1.11
Template:
C# Model:
Generated Typescript:
Expected Typescript:
The issues seem to be with any List<> in C#, and any nullable ints, it gets the nullable part right, but I can't see why it's generating the invalid int type.
The text was updated successfully, but these errors were encountered: