MagicMirror/serveronly/index.js

const app = require("../js/app");
const Log = require("../js/logger");

app.start().then((config) => {
	const bindAddress = config.address ? config.address : "localhost";
	const httpType = config.useHttps ? "https" : "http";
	Log.log(`\nReady to go! Please point your browser to: ${httpType}://${bindAddress}:${config.port}`);
});
Revise require imports (#3071) - order (external first) - remove superfluous file extensions - new line after imports - deconstruct (only one time (in `check_config.js`)) - fix path (only one time (in `global-setup.js`)) 2023-03-22 23:53:10 +01:00			`const app = require("../js/app");`
Fix import order in serveronly (#3072) To fix issue caused by #3071 In order to still fulfill the new linter rule `import/order`, I replaced the alias with the path. I also see two other approaches, but I opted for the simplest one here for now. The other approaches: 1. Also create an alias for `app`. 2. Use new `imports` (like in PR #2934), but let `_moduleAliases` untouched to stay compatible to 3rd party modules. Edit: Oh, I thought of another option: - Add `require("module-alias/register");` 2023-03-25 10:59:57 +01:00			`const Log = require("../js/logger");`
User logger in node files 2020-05-11 07:25:42 +02:00
Convert app-start/-stop callbacks to async/await (#3035) supersedes https://github.com/MichMich/MagicMirror/pull/3027 and just touches the start/stop calls. --------- Co-authored-by: veeck <michael@veeck.de> 2023-02-22 18:58:00 +01:00			`app.start().then((config) => {`
Use es6 notation in serveronly 2021-04-18 14:49:54 +02:00			`const bindAddress = config.address ? config.address : "localhost";`
			`const httpType = config.useHttps ? "https" : "http";`
Use template literals instead of string concatenation (#3066) We have used it inconsistently till now. Template literals are more modern and easier to maintain in my opinion. Because that's a large amount of changes, here's a way to reproduce it: I added the rule `"prefer-template": "error"` to the `.eslintrc.json` and did an autofix. Since this caused a new problem in line 409 of `newsfeed.js`, I reversed it in that line and also removed the rule from the eslint config file. The rule is described here: https://eslint.org/docs/latest/rules/prefer-template Note: I've played around with some other linter rules as well, and some seem to point to some specific, non-cosmetic, issues. But before I dive even deeper and then introduce even bigger and hardly understandable changes at once, I thought I'd start with this simple cosmetic rule. 2023-03-19 14:32:23 +01:00			Log.log(`\nReady to go! Please point your browser to: ${httpType}://${bindAddress}:${config.port}`);
Separate server core from Electron. 2016-04-08 22:16:22 +02:00			`});`