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

Display network configuration in the summary screen for NetworkManager #1198

Merged
merged 2 commits into from
Apr 8, 2021

Conversation

imobachgs
Copy link
Contributor

@imobachgs imobachgs commented Apr 8, 2021

Until now, YaST did not show the network configuration details in the summary screen when NetworkManager was the selected backend. Now, that's not the case anymore, and the network configuration is shown always.

Installation summary

@coveralls
Copy link

Coverage Status

Coverage remained the same at 79.651% when pulling 958491f on nm-proposal into 6981d43 on master.

Copy link
Contributor

@teclator teclator left a comment

Choose a reason for hiding this comment

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

LGTM

@imobachgs imobachgs merged commit b96f404 into master Apr 8, 2021
@imobachgs imobachgs deleted the nm-proposal branch April 8, 2021 14:22
@yast-bot
Copy link
Contributor

yast-bot commented Apr 8, 2021

✔️ Public Jenkins job #240 successfully finished
✔️ Created OBS submit request #883854

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.

None yet

4 participants