Skip to content

Claude/setup magicmirror 01 krvfdo jmo qs nfd r6pa4 ndk#3966

Closed
rxf-sys wants to merge 4 commits intoMagicMirrorOrg:masterfrom
rxf-sys:claude/setup-magicmirror-01KrvfdoJmoQsNfdR6pa4Ndk
Closed

Claude/setup magicmirror 01 krvfdo jmo qs nfd r6pa4 ndk#3966
rxf-sys wants to merge 4 commits intoMagicMirrorOrg:masterfrom
rxf-sys:claude/setup-magicmirror-01KrvfdoJmoQsNfdR6pa4Ndk

Conversation

@rxf-sys
Copy link
Copy Markdown

@rxf-sys rxf-sys commented Nov 17, 2025

Hello and thank you for wanting to contribute to the MagicMirror² project!

Please make sure that you have followed these 4 rules before submitting your Pull Request:

  1. Base your pull requests against the develop branch.
  2. Include these infos in the description:
  • Does the pull request solve a related issue?
  • If so, can you reference the issue like this Fixes #<issue_number>?
  • What does the pull request accomplish? Use a list if needed.
  • If it includes major visual changes please add screenshots.
  1. Please run node --run lint:prettier before submitting so that
    style issues are fixed.
  2. Don't forget to add an entry about your changes to
    the CHANGELOG.md file.

Note: Sometimes the development moves very fast. It is highly
recommended that you update your branch of develop before creating a
pull request to send us your changes. This makes everyone's lives
easier (including yours) and helps us out on the development team.

Thanks again and have a nice day!

rxf-sys and others added 4 commits November 17, 2025 15:55
Removed specific ignore rules for modules, CSS files, and config files.
CRITICAL SECURITY FIXES:
- Move all sensitive data (API keys, passwords) from config.js to separate credentials.js
- Add credentials.js to .gitignore to prevent accidental commits
- Create credentials.example.js as template for new installations
- Add comprehensive config/README.md with security documentation

BUG FIXES:
- Fix typo in OpenWeather module configuration (remove "NN" prefix)
- Disable debug mode in NewsAPI module

IMPROVEMENTS:
- Fix CSS variable references (--text-xs -> --font-size-xsmall, --text-sm -> --font-size-small)
- Improve code organization and documentation

This commit removes all sensitive credentials from version control while maintaining
functionality through a secure credentials import system.
@rxf-sys rxf-sys closed this Nov 17, 2025
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.

2 participants