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

PD-360 Update SCALE NFS Share Articles #1926

Merged
merged 3 commits into from
May 26, 2023

Conversation

micjohnson777
Copy link
Contributor

This commit updates the security information in the UI reference and AddingNFSShares.md article as requested.

It replaces the old image shortcode with the trueimage shortcode.

This change merges into Master as the content change applies to Bluefin in general.

Thanks for contributing to TrueNAS documentation! By opening a Pull Request, you're acknowledging that your changes will be distributed under the Creative Commons 4.0 license.

This commit updates the security information in the UI reference and AddingNFSShares.md article as requested.

It replaces the old image shortcode with the trueimage shortcode.
@micjohnson777 micjohnson777 requested a review from a team as a code owner May 24, 2023 19:48
@DjP-iX DjP-iX added the Technical Writer Review (DP) A technical writer is reviewing the PR. label May 26, 2023
Copy link
Contributor

@DjP-iX DjP-iX left a comment

Choose a reason for hiding this comment

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

Looks good. Just needs a fix to the inconsistent italics in lines 67 and 70

content/SCALE/SCALETutorials/Shares/AddingNFSShares.md Outdated Show resolved Hide resolved
removed the *root* emphasis.
@DjP-iX DjP-iX merged commit 8419aec into master May 26, 2023
3 checks passed
@DjP-iX DjP-iX deleted the PD-360-SCALE-Update-NFS-Share-Article-Content branch May 26, 2023 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Technical Writer Review (DP) A technical writer is reviewing the PR.
Projects
None yet
2 participants