Skip to content

Add documentation for running the prebuilt UI to the Getting Started guide#3702

Merged
sfeilmeier merged 2 commits into
OpenEMS:developfrom
mzenker:feature/guide-run-prebuilt-ui
May 4, 2026
Merged

Add documentation for running the prebuilt UI to the Getting Started guide#3702
sfeilmeier merged 2 commits into
OpenEMS:developfrom
mzenker:feature/guide-run-prebuilt-ui

Conversation

@mzenker
Copy link
Copy Markdown
Contributor

@mzenker mzenker commented Apr 28, 2026

Add a page describing how to run the prebuilt UI on Linux using the nginx web server. Link to that page from the Getting Started Guide. Small edits in the section "Run OpenEMS UI" of the Getting Started Guide.

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 30, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #3702      +/-   ##
=============================================
+ Coverage      60.16%   60.20%   +0.04%     
  Complexity       106      106              
=============================================
  Files           3234     3235       +1     
  Lines         141962   141999      +37     
  Branches       10446    10453       +7     
=============================================
+ Hits           85404    85483      +79     
+ Misses         53400    53347      -53     
- Partials        3158     3169      +11     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@da-Kai
Copy link
Copy Markdown
Contributor

da-Kai commented Apr 30, 2026

Hi @mzenker,

Thanks! You are right, the "how to deploy the UI"-Step is currently missing.

To keep our documentation consistent across the backend, edge, and ui, we should move this new file to the ui/deploy/ path? Something like ui/deploy/nginx.adoc would fit the existing structure.

@mzenker
Copy link
Copy Markdown
Contributor Author

mzenker commented May 1, 2026

Hi @da-Kai ,
the instructions I wrote are not really intended as instructions on how to deploy the UI, as this would include setting it up as a service and moving the UI files to a system location. I just wanted to describe a quick setup to fit with the instructions for running the prebuilt edge in the Getting Started Guide. So maybe I should rather include the contents of the new file ui/run-prebuilt.adoc directly in gettingstarted.adoc and create a file describing the "real" deployment of the UI as next step.

@da-Kai
Copy link
Copy Markdown
Contributor

da-Kai commented May 1, 2026

Hi @mzenker

I see your point that this is just a 'quick setup' and not a fully production-ready environment.
However, Setting up nginx to host the UI still falls under 'Deployment' for the end-user.

I would prefer to follow idiomatic design, consistent with most documentation.

Move this to ui/deploy/nginx.adoc and add a clear 'Quick Start' or 'Development Only' disclaimer at the top, noting that it's not intended for production use.
Later, we can add a 'Production' or 'Best Practices' section to that same file.

This keeps the gettingstarted.adoc from becoming a wall of text while ensuring the UI instructions live where people expect to find them.

@mzenker
Copy link
Copy Markdown
Contributor Author

mzenker commented May 1, 2026

Hi @da-Kai,
OK, I would then propose to move the "General" section of run-prebuilt.adoc to the Getting Started Guide and put the section "Set up the nginx webserver on Linux" in ui/deploy/nginx.adoc.

@mzenker
Copy link
Copy Markdown
Contributor Author

mzenker commented May 1, 2026

@da-Kai Done. Are the commits in this branch squashed together anyway when merged, or should I do it in the branch?

@da-Kai da-Kai requested a review from sfeilmeier May 3, 2026 19:37
Copy link
Copy Markdown
Contributor

@sfeilmeier sfeilmeier left a comment

Choose a reason for hiding this comment

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

Thank you!

@sfeilmeier sfeilmeier merged commit 2ff269d into OpenEMS:develop May 4, 2026
7 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.

3 participants