Merge lp:~clissold345/ubuntu-docs/minor-changes-to-firewall-page into lp:~ubuntu-core-doc/ubuntu-docs/trunk2017-07-10
Proposed by
Chris Perry
Status: | Merged |
---|---|
Merged at revision: | 564 |
Proposed branch: | lp:~clissold345/ubuntu-docs/minor-changes-to-firewall-page |
Merge into: | lp:~ubuntu-core-doc/ubuntu-docs/trunk2017-07-10 |
Diff against target: |
43 lines (+11/-8) 1 file modified
ubuntu-help/C/net-firewall-on-off.page (+11/-8) |
To merge this branch: | bzr merge lp:~clissold345/ubuntu-docs/minor-changes-to-firewall-page |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Gunnar Hjalmarsson | Approve | ||
Review via email: mp+309256@code.launchpad.net |
Description of the change
I've made some minor (mainly clarification) changes to net-firewall-
My main change is probably changing the page title. The old title was simply wrong in my opinion. What does "block firewall access" mean?
I've corrected a few inconsistencies and made a few clarifications. Perhaps my only contentious change is to mention that other firewalls are available?
To post a comment you must log in.
Looks good, thanks for your work.
I made a tiny change when merging: Surrounding SSH with <app></app> seems not to be right, since SSH in this context is reasonably the protocol rather than a specific application, so I dropped those.
You can find out about the meaning of various inline elements at <http:// projectmallard. org/1.0/>.