MarimerLLC
WPFUi should use appsettings.json for DataPortal URL
To get better jumpstart on CSLA using WPF Ui, I have made adjustments to ProjectTracker where WPFUi is using appsettings.json.
Hope to do the same for Blazor Client, but maybe it's better to wait for final release.
If it's ok, and I have the right to do so, I would make an Pull request.
@BlagoCuljak this new set of changes (https://github.com/MarimerLLC/csla/issues/1548) will impact your PR I think.
We can undo it, if we're about to have more issues.
Didn't look too much into the issue, I'm on mobile app.
@BlagoCuljak Do you want to revisit this now that the dust has settled on the other changes around DI and app initialization?
I do think having config values read correctly would be a great step forward!
Sure, what do you propose? That we scrap these changes in favor of #1548?
Merge your reading of the config file/settings into the new (current) app config code.
https://github.com/MarimerLLC/csla/blob/master/Samples/ProjectTracker/ProjectTracker.Ui.WPF/App.xaml.cs
Dude, you always get me on some holiday. 😀 First Easter, now Labour day. If you don't want to wait for next week, could you please do it yourself? Thanks
There's no rush, I was just wondering if you remained interested in working on this.
Rocky, could you do the merge, please? I'm having issues that prevent me from doing so, sorry.
You mean for the original PR?
https://github.com/MarimerLLC/csla/pull/1543
I think the problem is that PR changed files (mostly Program.cs) that have been radically altered since the PR was created. It may not be possible to merge at this point.
It is almost certainly easier to take the current Program.cs, which uses the new UsingCsla method coming in 5.2.0, and add the code to read the config settings using the standard .NET config subsystem.