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

Communication docs: note that editing Matrix messages causes IRC spam #79238

Merged
merged 2 commits into from Oct 27, 2022

Conversation

felixfontein
Copy link
Contributor

SUMMARY

The current wording apparently encourages folks to edit their messages, while they're not being aware that evey edit creates another copy of the message for IRC users (sometimes just parts of it, but often enough the full message is sent again).

CC @GregSutcliffe

ISSUE TYPE
  • Docs Pull Request
COMPONENT NAME

docs/docsite/rst/community/communication.rst

@ansibot
Copy link
Contributor

ansibot commented Oct 27, 2022

Thanks for your Ansible docs contribution! We talk about Ansible documentation on matrix at #docs:ansible.im and on libera IRC at #ansible-docs if you ever want to join us and chat about the docs! We meet there on Tuesdays (see the Ansible calendar) and welcome additions to our weekly agenda items - scroll down to find the upcoming agenda and add a comment to put something new on that agenda.

click here for bot help

@ansibot ansibot added WIP This issue/PR is a work in progress. Nevertheless it was shared for getting input from peers. affects_2.15 docs This issue/PR relates to or includes documentation. docs_only All changes are to files within the docs/docsite/ directory docsite This issue/PR relates to the documentation website. needs_triage Needs a first human triage before being processed. small_patch labels Oct 27, 2022
@GregSutcliffe
Copy link
Contributor

GregSutcliffe commented Oct 27, 2022

LGTM, thanks @felixfontein & @oraNod!

Co-authored-by: Don Naro <dnaro@redhat.com>
@felixfontein felixfontein marked this pull request as ready for review October 27, 2022 09:35
@ansibot ansibot removed the WIP This issue/PR is a work in progress. Nevertheless it was shared for getting input from peers. label Oct 27, 2022
@oraNod oraNod merged commit bc83da9 into ansible:devel Oct 27, 2022
@oraNod
Copy link
Contributor

oraNod commented Oct 27, 2022

Thanks @felixfontein

@oraNod oraNod removed the needs_triage Needs a first human triage before being processed. label Oct 27, 2022
@ansibot ansibot added needs_rebase https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. labels Oct 27, 2022
@felixfontein felixfontein deleted the matrix-edits branch October 27, 2022 10:25
felixfontein added a commit to felixfontein/ansible that referenced this pull request Oct 27, 2022
felixfontein added a commit to felixfontein/ansible that referenced this pull request Oct 27, 2022
sivel pushed a commit that referenced this pull request Oct 27, 2022
sivel pushed a commit that referenced this pull request Oct 27, 2022
@ansible ansible locked and limited conversation to collaborators Nov 3, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
affects_2.15 docs_only All changes are to files within the docs/docsite/ directory docs This issue/PR relates to or includes documentation. docsite This issue/PR relates to the documentation website. needs_rebase https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. small_patch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants