passcore icon indicating copy to clipboard operation
passcore copied to clipboard

Further improvements to support .NET 6 migration

Open unquietwiki opened this issue 3 years ago • 7 comments

Hey @geoperez and crew. I had to re-install this for one of my employers, and used the opportunity to see if there was anything I was running into. I have yesterday's changes running fine; today's stuff is the start of some refactoring, which you might want to review before doing any more. Part of that may help with an issue I've run into, where a failed new password (because of requirements on SAMBA/AD), comes up as an incorrect current password error. I also ended up running this without Docker again; if there was an easier way to separate out & map the JSON config for that, I think it'd definitely help that use.

Hope you all are keeping safe & well.

unquietwiki avatar Jun 16 '22 21:06 unquietwiki

@geoperez Also, I dunno if you can update your AppVeyor config, but it's failing on .NET 6 builds; seems like it's using VS 2017 stuff.

unquietwiki avatar Jun 16 '22 21:06 unquietwiki

Hi @unquietwiki , quick question. Why do you change from "var" to type in variable declarations?

I'll check the Appveyor thing.

geoperez avatar Jun 16 '22 21:06 geoperez

Hi @unquietwiki , quick question. Why do you change from "var" to type in variable declarations?

I'll check the Appveyor thing.

Yeah, I was hesitant on the var-change; but VS kept suggesting it, and it was coming up with suggestions that exposed the nullable types, so that seemed more intuitive. A lot of the stuff SonarLint flagged relates to either nulls, or reference issues.

unquietwiki avatar Jun 16 '22 21:06 unquietwiki

AppVeyor settings have been updated.

geoperez avatar Jun 17 '22 13:06 geoperez

The front-end build is failing.

geoperez avatar Jun 17 '22 13:06 geoperez

@geoperez Managed to catch me up early; cat woke me up. I re-added react as a direct-dependency, even though the Uno stuff loads it. Can run build on my machine just fine; not sure why it's failing in Appveyor. Both my machine & my other install are using Node 16.x, if that helps.

unquietwiki avatar Jun 17 '22 13:06 unquietwiki

Let me try to test it on my local env.

geoperez avatar Jun 17 '22 14:06 geoperez

Hi, any updates regarding .NET Core Version? We wanna get rid of old unsupported .NET Core Version.

VM-Master avatar Feb 08 '23 10:02 VM-Master