Skip to content

Conversation

Copy link

Copilot AI commented Nov 24, 2025

Addresses review feedback from #131 where validation error messages referenced undefined translation keys.

Changes:

  • Added error.tun_mtu_min and error.mssfix_min to ui/public/i18n/en/translation.json

The Settings.vue validation code was already checking for minimum values (576 for TUN MTU, 0 for MSS Fix) but the corresponding translation strings were missing, causing runtime errors when validation failed.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Update mtu and mssfix options based on review feedback Add missing translation keys for MTU/MSS Fix validation errors Nov 24, 2025
Copilot AI requested a review from gsanchietti November 24, 2025 10:42
@gsanchietti gsanchietti marked this pull request as ready for review November 24, 2025 10:54
@gsanchietti gsanchietti merged commit d9d3f13 into mtu_option Nov 24, 2025
@gsanchietti gsanchietti deleted the copilot/sub-pr-131 branch November 24, 2025 10:54
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