Remove code from Main.brs - create App class to handle init, login flow, and main loop

[cewert]

Restructure Main.brs to use the least amount of code as possible because nothing in Main.brs can be unit tested and also because roku-log won't work in Main.brs. I did keep some of the global stuff in Main.brs because it needs to be run as early as possible and also because there's only so much we can do while the "splashscreen" is up.

To do this I created a new class called App.bs which is used to initialize the app, handle the login flow, and the main loop(with m.port). roku-log is saved to the log var and can be accessed with m.log when running code inside the class (when inside a class m refers to the class members and methods).

I've also updated the LoginFlow() function to use roku-log. To do this I imported the roku-log file at the top of ShowScenes.brs and setup a logger inside the function. This isn't ideal since every function would have to setup a logger but was needed because of all the debug statements inside LoginFlow().

Changes

• Update bsconfig.json - don't remove roku-log statements for dev builds (I thought these were false by default but apparently not. This is why I wasn't able to get this working originally)
• Move roku-log setup from JFScreen.brs to JFScene.brs
• Remove as much code from Main.brs as possible
• Create App class to handle init, login flow, and main app loop
• Setup roku-log inside the App class
• Update code to work inside class
• Update all current print statements to use roku-log(within reason. I didn't import roku-log into source files that only had one print statement)
• Add some log statements where needed

NOTE: I did NOT convert the print statement that spits out the array of user settings because when roku-log prints associative arrays they are very ugly. using print looks much better 😄

Issues

Fixes #1245

Still working on a few things so marking this as draft for now

[jellyfin-bot]

This pull request has merge conflicts. Please resolve the conflicts so the PR can be reviewed. Thanks!

[candry7731]

Let me know if you need help testing any of this.

[jellyfin-bot]

This pull request has merge conflicts. Please resolve the conflicts so the PR can be reviewed. Thanks!

[jellyfin-bot]

This pull request has merge conflicts. Please resolve the conflicts so the PR can be reviewed. Thanks!

[jellyfin-bot]

This pull request has merge conflicts. Please resolve the conflicts so the PR can be reviewed. Thanks!