Type.ToTypeScriptType Generating Incorrectly about ntypewriter HOT 7 CLOSED

DateTime? also seems to be generating DateTime<DateTime> | null and importing DateTime as a custom type.

from ntypewriter.

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?

from ntypewriter.

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.

from ntypewriter.

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.

from ntypewriter.

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...

from ntypewriter.

@vcastellano-spl congrats for diving into the problem and helping out the maintainer of the project, not something many people do!
Also maybe it's a typo, but you should use ; instead of , in the TS interface definition to separate the properties.

from ntypewriter.

@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 ;.

from ntypewriter.