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 wireless mode selection #5583

Merged
merged 1 commit into from
Nov 27, 2024

Conversation

pierantoniomerlino
Copy link
Contributor

Note: We are using the Conventional Commits convention for our pull request titles. Please take a look at the PR title format document for the supported types and scopes.

This PR fixes a weird behavior in the Wireless tab when the Enable for WAN is selected.

Related Issue: This PR fixes/closes N/A

Description of the solution adopted: When the Wireless tab is selected and the interface is configured as Enabled for WAN and DHCP, the Station Mode is not selectable. This PR fixes this wrong behavior. The cause of this is a wrong enum comparison.

BTW, the release-5.6.0 contains the wrong comparison, while the Kura 5.6.0 tag doesn't.

Signed-off-by: pierantoniomerlino <[email protected]>
@MMaiero MMaiero merged commit d4b7cfd into develop Nov 27, 2024
4 checks passed
@MMaiero MMaiero deleted the fix_wireless_mode_selection branch November 27, 2024 08:21
eclipse-kura-bot pushed a commit that referenced this pull request Nov 27, 2024
Fixed wireless mode selection

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

fix(web2): Fixed wireless mode selection (#5583)

Fixed wireless mode selection

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

Co-authored-by: Pierantonio Merlino <[email protected]>
MMaiero pushed a commit that referenced this pull request Dec 20, 2024
Fixed wireless mode selection

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.

2 participants