Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(web2): Fixed MTU field visualization #5551

Merged
merged 1 commit into from
Nov 13, 2024
Merged

Conversation

pierantoniomerlino
Copy link
Contributor

This PR fixes the MTU field visualization for both IPv4 and IPv6.

Related Issue: This PR fixes/closes N/A

Description of the solution adopted: A GWT configuration is added to remove the visualization of the comma (,) as thousands separator in the MTU fields. Moreover, the MTU value check has been improved.

@MMaiero
Copy link
Contributor

MMaiero commented Nov 13, 2024

Forcing merge. The changes will be tested in the ongoing QA testing process

@MMaiero MMaiero merged commit 68a2b0e into develop Nov 13, 2024
2 of 3 checks passed
@MMaiero MMaiero deleted the fix_mtu_web_field branch November 13, 2024 12:44
eclipse-kura-bot pushed a commit that referenced this pull request Nov 13, 2024
Changed number style; updated integer validation

Signed-off-by: pierantoniomerlino <[email protected]>
(cherry picked from commit 68a2b0e)
MMaiero pushed a commit that referenced this pull request Nov 13, 2024
)

fix(web2): Fixed MTU field visualization (#5551)

Changed number style; updated integer validation

Signed-off-by: pierantoniomerlino <[email protected]>
(cherry picked from commit 68a2b0e)

Co-authored-by: Pierantonio Merlino <[email protected]>
MMaiero pushed a commit that referenced this pull request Dec 20, 2024
Changed number style; updated integer validation

Signed-off-by: pierantoniomerlino <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants