Skip to content

UI: Add IPv6 support - #3689

Open
da-Kai wants to merge 17 commits into
OpenEMS:developfrom
da-Kai:feature/ui-dev/ipv6
Open

UI: Add IPv6 support#3689
da-Kai wants to merge 17 commits into
OpenEMS:developfrom
da-Kai:feature/ui-dev/ipv6

Conversation

@da-Kai

@da-Kai da-Kai commented Apr 21, 2026

Copy link
Copy Markdown
Contributor

Add IPv6 capabilities for UI
Move Network related regex pattern and validation functions into a separate InetUtils-namespace.
Network settings: Keep IPv4 validation, until edge supports IPv6

Address #3484 partially

@da-Kai
da-Kai requested a review from lukasrgr April 21, 2026 20:24
@codecov

codecov Bot commented Apr 21, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 91.07143% with 5 lines in your changes missing coverage. Please review.

Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #3689      +/-   ##
=============================================
+ Coverage      55.62%   55.64%   +0.03%     
+ Complexity     19017    19016       -1     
=============================================
  Files           3670     3671       +1     
  Lines         171475   171528      +53     
  Branches       14127    14145      +18     
=============================================
+ Hits           95362    95432      +70     
+ Misses         72337    72317      -20     
- Partials        3776     3779       +3     
🚀 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.

@github-actions

Copy link
Copy Markdown

This PR has been automatically marked as stale due to inactivity

If it is still relevant, please update the PR.
Otherwise, it will be automatically closed in 14 days.

Before updating, please review our Contribution Guidelines to ensure all required information is provided.

@github-actions github-actions Bot added the Stale label Jul 27, 2026
@sfeilmeier

Copy link
Copy Markdown
Contributor

@lukasrgr, @DerMuaaa, @venu-sagar Could anyone of you have a look please?

@venu-sagar venu-sagar left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Solid refactor. regexes are well-known and tested.

Comment thread ui/src/app/shared/utils/inet/inet.utils.ts Outdated
Comment thread ui/src/app/shared/utils/inet/inet.utils.ts Outdated
Comment thread ui/src/app/shared/utils/inet/inet.utils.ts Outdated
Comment thread ui/src/app/shared/utils/inet/inet.utils.spec.ts Outdated
* uniform naming
* extend tests
* cleanup
@da-Kai
da-Kai requested review from sfeilmeier and removed request for lukasrgr August 19, 2026 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants