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-1252 / 25.04 / Pd 1252 update/replace smb home shares content #3158

Open
wants to merge 22 commits into
base: master
Choose a base branch
from

Conversation

micjohnson777
Copy link
Contributor

@micjohnson777 micjohnson777 commented Oct 24, 2024

This PR can backport to 24.10 and 24.04

This PR updates the content in the SMB Shares UI ref and tutorials in /sharing/smb/_index.md and AddSMBHomeShare.md articles.
Content updates cover the alternative method to creating home shares.

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 /shares/SMB/_index.md file with new information on the Private SMB Datasets and Shares preset, fixes style issues, and clarifies other content.
It makes initial and minor edits to the Home Shares article
This commit updates the content and screenshots in the AddSMBHomeShare.md tutorial to show setting up the dataset, share, ACL preset, and adding the user home directory to use the *private* dataset, which is the alternative to using home shares.
@micjohnson777 micjohnson777 requested a review from a team as a code owner October 24, 2024 19:25
@micjohnson777 micjohnson777 marked this pull request as draft October 24, 2024 19:25
@bugclerk bugclerk changed the title Pd 1252 update/replace smb home shares content PD-1252 / 25.04 / Pd 1252 update/replace smb home shares content Oct 24, 2024
@bugclerk
Copy link
Contributor

This commit updates the Other Options screenshot and adds Additional Parameters String to the list of setting options.
This commit adds the close expand shortcode to the tutorial /shares/smb/_index.md file.
@micjohnson777 micjohnson777 marked this pull request as ready for review October 24, 2024 20:56
@micjohnson777
Copy link
Contributor Author

Requesting a review by Andrew to verify the procedure and settings included are correct, and for general feedback on the changes to the AddSMBHomeShare.md article.

content/SCALE/SCALETutorials/Shares/SMB/AddSMBHomeShare.md Outdated Show resolved Hide resolved
content/SCALE/SCALETutorials/Shares/SMB/AddSMBHomeShare.md Outdated Show resolved Hide resolved
content/SCALE/SCALETutorials/Shares/SMB/AddSMBHomeShare.md Outdated Show resolved Hide resolved
content/SCALE/SCALETutorials/Shares/SMB/AddSMBHomeShare.md Outdated Show resolved Hide resolved
content/SCALE/SCALETutorials/Shares/SMB/AddSMBHomeShare.md Outdated Show resolved Hide resolved
content/SCALE/SCALETutorials/Shares/SMB/AddSMBHomeShare.md Outdated Show resolved Hide resolved
content/SCALE/SCALETutorials/Shares/SMB/AddSMBHomeShare.md Outdated Show resolved Hide resolved
content/SCALE/SCALETutorials/Shares/SMB/_index.md Outdated Show resolved Hide resolved
@DjP-iX DjP-iX added the Technical Writer Review (DP) A technical writer is reviewing the PR. label Oct 29, 2024
#3173)

* Update _index.md

* Update SMBSharesScreens.md

* Create SMBPurposePresets.md
@DjP-iX
Copy link
Contributor

DjP-iX commented Nov 1, 2024

Changes look good! Soft approved, but want to get SME feedback before we merge it.

@DjP-iX DjP-iX removed the Re-Review label Nov 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants