Skip to content

Update Nginx Proxy Manager docs for the UI rewrite - #67

Merged
GilbN merged 1 commit into
developfrom
docs/npm-ui-707
Sep 14, 2026
Merged

GilbN merged 1 commit into
developfrom
docs/npm-ui-707

Conversation

@GilbN

@GilbN GilbN commented Sep 14, 2026

Copy link
Copy Markdown
Collaborator

Update NPM documentation for the React UI rewrite in themepark-dev/theme.park#734. Document version compatibility, correct the startup-script mount path, and explain theme settings and container recreation.

Replace the old gallery with NPM 2.15.1 captures for all 11 official options, including the missing Maroon image. MkDocs builds successfully with existing configuration warnings; the rendered NPM page and all gallery images were checked in Chromium.

Publish alongside the theme rewrite so the documented compatibility matches the hosted CSS.

Organizr gallery comparison. The previous image shows the legacy UI; its exact NPM version is unknown. The replacement shows NPM 2.15.1.

Previous gallery image Updated gallery image
Previous NPM gallery Updated NPM gallery

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

No unresolved issues were identified.

Pull request overview

Updates Nginx Proxy Manager documentation for the React UI rewrite.

Changes:

  • Documents version compatibility and NPM 2.15.1 validation.
  • Corrects startup-script mounting and explains configuration and recreation.
  • Refreshes gallery context for the official themes.
File summaries
File Summary
docs/themes/nginx-proxy-manager.md Updated compatibility, Docker setup, configuration, and gallery guidance.
Review details
  • Files reviewed: 1/12 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@GilbN
GilbN merged commit be4500e into develop Sep 14, 2026
5 checks passed
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