Skip to content

Few changes made on the Smartmirror Project#20

Closed
3DJupp wants to merge 45 commits into
MagicMirrorOrg:masterfrom
3DJupp:master
Closed

Few changes made on the Smartmirror Project#20
3DJupp wants to merge 45 commits into
MagicMirrorOrg:masterfrom
3DJupp:master

Conversation

@3DJupp

@3DJupp 3DJupp commented May 25, 2015

Copy link
Copy Markdown

Translation (NE,DL,FR,EN etc) compliments,moments and all other items
Language override (if browser locale is wrong) just enter http://smartmirror/?en or de
Fixed .1f Temperatures
Support for multiple Calendars (just config that in calendar.php)
Updated:moment.js and Temperature icons

@3DJupp 3DJupp closed this Nov 24, 2015
MichMich pushed a commit that referenced this pull request Apr 23, 2016
khassel pushed a commit that referenced this pull request May 12, 2026
Small cleanup PR to close three CodeQL alerts, without changing app
behavior.

-
[#18](https://github.com/MagicMirrorOrg/MagicMirror/security/code-scanning/18)
in `js/loader.js`
-
[#19](https://github.com/MagicMirrorOrg/MagicMirror/security/code-scanning/19)
in `js/socketclient.js`
-
[#20](https://github.com/MagicMirrorOrg/MagicMirror/security/code-scanning/20)
in `tests/electron/modules/calendar_spec.js`

For `Loader` and `MMSocket`, make the global export explicit via
`globalThis` (instead of suppressing `no-unused-vars`).
For calendar tests, remove the unused debug helper `logAllText`.
Also includes a tiny e2e cleanup: `MM_PORT` is cleared in `afterAll`.

Outcome: Three open CodeQL alerts are addressed with small, low-risk
changes. The global intent for `Loader` and `MMSocket` is explicit, dead
test helper code is removed, and e2e test state is cleaned up more
reliably between runs.

With this, all current [code scanning
issues](https://github.com/MagicMirrorOrg/MagicMirror/security/code-scanning)
should be resolved in the develop branch 🎈
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant