Skip to content

docs: use folder terminology in mail manual#14860

Merged
skjnldsv merged 2 commits into
nextcloud:masterfrom
MukundaKatta:codex/rename-mailbox-folder-docs
May 19, 2026
Merged

docs: use folder terminology in mail manual#14860
skjnldsv merged 2 commits into
nextcloud:masterfrom
MukundaKatta:codex/rename-mailbox-folder-docs

Conversation

@MukundaKatta
Copy link
Copy Markdown
Contributor

Summary

  • update the Mail user manual to use folder terminology instead of mailbox terminology
  • keep the existing shared mailbox icon asset reference intact while updating visible text

Fixes #13898

Testing

  • git diff --check
  • verified referenced shared mailbox icon file exists

Note: full Sphinx build was not run locally because Sphinx is not installed in this environment.

@MukundaKatta
Copy link
Copy Markdown
Contributor Author

Pushed 310676b to fix the DCO failure.

Root cause: the single PR commit was missing the required Signed-off-by trailer. I amended the commit with signoff and force-pushed with lease. The documentation diff itself is unchanged.

Validation run locally:

  • git diff --check
  • verified the amended commit contains Signed-off-by

Copy link
Copy Markdown
Member

@ChristophWurst ChristophWurst left a comment

Choose a reason for hiding this comment

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

The text and the subline of a heading has to match in length

Please build the documentation locally and check any warnings and errors

Also post a screenshot of the HTML manual with your applied changes

Thanks for the contribution

@skjnldsv skjnldsv requested a review from ChristophWurst May 19, 2026 10:50
MukundaKatta and others added 2 commits May 19, 2026 12:51
Signed-off-by: mukunda katta <mukunda.vjcs6@gmail.com>
Heading underlines must match text length. The mailbox→folder rename
shortened 7 headings but left underlines at old length.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
@skjnldsv skjnldsv enabled auto-merge May 19, 2026 10:51
@skjnldsv skjnldsv force-pushed the codex/rename-mailbox-folder-docs branch from 5b4b5ef to 6f48063 Compare May 19, 2026 10:51
@skjnldsv skjnldsv merged commit 36bd5fd into nextcloud:master May 19, 2026
23 of 25 checks passed
@welcome
Copy link
Copy Markdown

welcome Bot commented May 19, 2026

Thanks for your first pull request and welcome to the community! Feel free to keep them coming! If you are looking for issues to tackle then have a look at this selection: https://github.com/nextcloud/documentation/issues?q=is%3Aopen+is%3Aissue+label%3A%22good+first+issue%22

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.

Rename Mailbox to folder in Mail documentation

3 participants