search

opnsense / docs

OPNsense documentation
Other
114 stars 181 forks source link

Zenarmor patch 3 #509

Closed zenarmor closed 10 months ago

zenarmor commented 10 months ago

I have totally updated [zenarmor_install.rst] file according to the new Zenarmor version 1.15

AdSchellevis commented 10 months ago

@zenarmor there are some build warnings and errors in the PR, see below for examples.

zenarmor_install.rst:86: ERROR: Content block expected for the "Note" directive; none found.
zenarmor_install.rst:137: WARNING: Title overline too short.

Most are easy to fix, is this all you want to update at the moment? I can fix the errors for you, but rather do it all at once.

zenarmor commented 10 months ago

If you can update, I would be very happy. Yes, for now, I'm updating this part, and I plan to work on and send the other pages on a separate branch.

On Wed, Oct 25, 2023 at 10:51 PM Ad Schellevis @.***> wrote:

@zenarmor https://github.com/zenarmor there are some build warnings and errors in the PR, see below for examples.

zenarmor_install.rst:86: ERROR: Content block expected for the "Note" directive; none found. zenarmor_install.rst:137: WARNING: Title overline too short.

Most are easy to fix, is this all you want to update at the moment? I can fix the errors for you, but rather do it all at once.

— Reply to this email directly, view it on GitHub https://github.com/opnsense/docs/pull/509#issuecomment-1779951469, or unsubscribe https://github.com/notifications/unsubscribe-auth/A4AB7FOU5RDJK5HIVJ3B3ALYBFUSJAVCNFSM6AAAAAA6P5M3E6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZZHE2TCNBWHE . You are receiving this because you were mentioned.Message ID: @.***>

AdSchellevis commented 10 months ago

@zenarmor ok, I'll merge and fix. please try a "make html" on your end for the next batch.

zenarmor commented 10 months ago

Thanks, I will.

On Wed, Oct 25, 2023 at 11:10 PM Ad Schellevis @.***> wrote:

@zenarmor https://github.com/zenarmor ok, I'll merge and fix. please try a "make html" on your end for the next batch.

— Reply to this email directly, view it on GitHub https://github.com/opnsense/docs/pull/509#issuecomment-1779980551, or unsubscribe https://github.com/notifications/unsubscribe-auth/A4AB7FJLJJ7BFX5RRDX2E4LYBFW4LAVCNFSM6AAAAAA6P5M3E6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZZHE4DANJVGE . You are receiving this because you were mentioned.Message ID: @.***>

AdSchellevis commented 10 months ago

https://docs.opnsense.org/vendor/sunnyvalley/zenarmor_install.html