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
Allow to change the default zone #85
Conversation
82dcccd
to
3634e8b
Compare
53e7039
to
ff1cf23
Compare
ff1cf23
to
09afc83
Compare
end | ||
|
||
# @see CWM::Table#header | ||
def header | ||
[ | ||
_("Name"), | ||
_("Interfaces") | ||
_("Interfaces"), | ||
_("Default") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure about having an extra column for it, but probably that is something for Ken, I would maybe prefer something like public (default)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We have something similar in the repository list, but it is a slightly different case. But we might append a "(default)" string if preferred.
LGTM once travis is fine with the rdoc check 😉 |
Thanks :) |
This PR allows to change the Firewalld's default zone.