Skip to content

NAS-137780 / 27.0.0-BETA.1 / UX improvements for the interface configuration workflow#13272

Draft
aervin wants to merge 3 commits intomasterfrom
NAS-137780
Draft

NAS-137780 / 27.0.0-BETA.1 / UX improvements for the interface configuration workflow#13272
aervin wants to merge 3 commits intomasterfrom
NAS-137780

Conversation

@aervin
Copy link
Copy Markdown
Contributor

@aervin aervin commented Feb 24, 2026

Changes:

  • draw more attention to interface configuration content (Test Changes, Save Changes, etc)
  • proactive approach to preventing users from removing a static IP if the web UI is bound to that address
  • provide a "Test and Open UI" option when convenient

Testing:

  • test interface configuration when the UI bind address is a wildcard
  • test interface configuration when the UI bind address is explicitly set
Screenshot 2026-02-24 at 3 01 44 PM Screenshot 2026-02-24 at 3 02 17 PM Screenshot 2026-02-24 at 2 28 12 PM Screenshot 2026-02-24 at 2 52 18 PM

Downstream

Affects Reasoning
Documentation The "test network configuration" workflow is largely unchanged, but the content is different visually now.
Testing A new button is sometimes available: Test and Open UI (ixTest="test-and-open-ui")

@aervin aervin self-assigned this Feb 24, 2026
@bugclerk bugclerk changed the title UX improvements for the interface configuration workflow NAS-137780 / 26.0.0-BETA.1 / UX improvements for the interface configuration workflow Feb 24, 2026
@bugclerk
Copy link
Copy Markdown
Contributor

@aervin
Copy link
Copy Markdown
Contributor Author

aervin commented Feb 24, 2026

time 3:00

@claude
Copy link
Copy Markdown

claude Bot commented Feb 24, 2026

Claude encountered an error —— View job


I'll analyze this and get back to you.

@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 24, 2026

Codecov Report

❌ Patch coverage is 73.63636% with 29 lines in your changes missing coverage. Please review.
✅ Project coverage is 86.60%. Comparing base (e820424) to head (75ce72e).
⚠️ Report is 10 commits behind head on master.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/app/pages/system/network/network.component.ts 68.75% 25 Missing ⚠️
...ponents/interface-form/interface-form.component.ts 86.66% 4 Missing ⚠️

❌ Your patch check has failed because the patch coverage (73.63%) is below the target coverage (75.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #13272      +/-   ##
==========================================
- Coverage   86.63%   86.60%   -0.03%     
==========================================
  Files        1871     1871              
  Lines       71799    71917     +118     
  Branches     8336     8367      +31     
==========================================
+ Hits        62206    62287      +81     
- Misses       9593     9630      +37     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@bugclerk bugclerk changed the title NAS-137780 / 26.0.0-BETA.1 / UX improvements for the interface configuration workflow NAS-137780 / 26.0.0-BETA.2 / UX improvements for the interface configuration workflow Mar 4, 2026
@bugclerk bugclerk changed the title NAS-137780 / 26.0.0-BETA.2 / UX improvements for the interface configuration workflow NAS-137780 / 27.0.0-BETA.1 / UX improvements for the interface configuration workflow Mar 4, 2026
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