search

mbbsemu / MBBSEmu

The MajorBBS Emulation Project is an Open Source, Cross-Platform emulator for easily running The MajorBBS & Worldgroup Modules
https://www.mbbsemu.com
MIT License
126 stars 14 forks source link

MBBSEMU - Changes in setup wizard are not reflected in appsettings.json #605

Closed Purduecoz closed 6 months ago

Purduecoz commented 7 months ago

MBBSEMU Stable Alpha v1.0-alpha-112423

Making changes to the default options in the new MBBSEmu GUI wizard does not reflect the changes made either in the initial setup wizard review or when the appsettings.json file is written.

To reproduce, change simple items such as BBS Title, Registration number, or telnet port

Expected behavior - the options changes are written to appsettings.json

Behavior - the default options are still written

 Software Information:

enusbaum commented 7 months ago

I've seen this issue crop up with the "Self-Contained" EXE files that dotnet packages with not properly reflecting pwd

I think I'll add the "file saved to" or perhaps a save dialogue since I think pwd for the packaged EXE is a temp folder, and not the application directory.

enusbaum commented 7 months ago

In #606 I added a file dialogue at the end of setup and a message box confirming the path.

image image

This should help us around the pwd issue for now.

enusbaum commented 6 months ago

Fixed with #606