-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Adding popup messages for errors with status codes alongside the exception message #2116
Conversation
… NitroxModel for the messageBox and NitroxServer for the console log(improves efficiency)
…erver status codes into NitroxModel file
Pull from indev, base function is done, just need to add tests and fix any potential bugs
…code for NitroxServer
… messageBox's width, expanded messageBox vertically
New codes as of latest commit: |
…test and then it's ready for review
… for review after I playtest
…loaded in NitroxPatcher
Merge latest nitrox changes into my PR
…abruptly, don't know why I added this earlier, honestly should've realized this sooner)
…less important codes in release. The idea here is that popups are only displayed for status codes that would be standalone and important, entirely disrupting the user experience
So many current PRs would be incompatible with this and it's not really helpful so I think I will probably be closing this PR. Will give it some more thought / mourn my time before I click the button though. |
had to be done, rest in peace PR you will most certainly be forgotten and not remembered |
Add the following status codes that appear in a popup window when nitrox crashes or encounters an error. The exception is shown in the message alongside one of the following codes: