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

Added communication principles #1168

Closed
wants to merge 4 commits into from
Closed

Added communication principles #1168

wants to merge 4 commits into from

Conversation

BhaswatiRoy
Copy link

@BhaswatiRoy BhaswatiRoy commented Jan 25, 2023

Fix: #104

Communication principles are added to the "contributors" section in Mattermost Handbook.

(Being a first-time contributor, I have signed the Mattermost CLA)

@mattermost-build
Copy link

Hello @BhaswatiRoy,

Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

@justinegeffen justinegeffen self-requested a review January 25, 2023 13:16
# Communication Principles

As we share ideas and communicate within the community, we want to ensure that communication principles are followed.
You can have a look at the set within the Mattermost Community, there are instructions that fall under the Communication Principle.
Copy link
Contributor

Choose a reason for hiding this comment

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

Do you have a link to this set of principles or the Communication Principle? I think it would be helpful to link it here. :)

Copy link
Author

Choose a reason for hiding this comment

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

Hello @justinegeffen
I actually wrote about the principles mentioned in this issue itself
#104

Copy link
Contributor

Choose a reason for hiding this comment

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

Thank you! I don't see any instructions on that issue and I'm not aware of a set documented anywhere. I propose removing this line, especially if it refers to the bullet points that follow. :)

@justinegeffen
Copy link
Contributor

Thanks for this PR, @BhaswatiRoy! I'm adding @jordanafung to the review process as this is related to community content.

Having looked through the contributors section, I wonder if the content of this PR wouldn't work better rolled up into this page: https://handbook.mattermost.com/contributors/contributors/guidelines/social-engagement-guidelines as a new section?

@BhaswatiRoy
Copy link
Author

BhaswatiRoy commented Jan 25, 2023

Yeah ig it can be added there too!
should I make the required changes?

@mattermost-build
Copy link

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!

@justinegeffen
Copy link
Contributor

Thank you for your contribution, @BhaswatiRoy. I'm closing this PR as it requires more investigation into the intended content of the original issue.

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.

Discussion: Document communication principles
3 participants