mattermost-developer-documentation icon indicating copy to clipboard operation
mattermost-developer-documentation copied to clipboard

Added workflow diagrams

Open Drishti-jain21 opened this issue 3 years ago • 8 comments

Summary

I have added 3 workflow diagrams to summarize the You've found a bug content under Why-Contribute.

Ticket Link

#1143

Drishti-jain21 avatar Oct 14 '22 13:10 Drishti-jain21

Hello @Drishti-jain21,

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.

Per the Mattermost Contribution Guide, we need to add you to the list of approved contributors for the Mattermost project.

Please help complete the Mattermost contribution license agreement? Once you have signed the CLA, please comment with /check-cla and confirm that the CLA check is green.

This is a standard procedure for many open source projects.

Please let us know if you have any questions.

We are very happy to have you join our growing community! If you're not yet a member, please consider joining our Contributors community channel to meet other contributors and discuss new opportunities with the core team.

mattermod avatar Oct 14 '22 13:10 mattermod

Newest code from mattermod has been published to preview environment for Git SHA 5a4fc4faa7936e7154485b123f56e353dfdf44ec

github-actions[bot] avatar Oct 14 '22 13:10 github-actions[bot]

Newest code from mattermod has been published to preview environment for Git SHA 5a4fc4faa7936e7154485b123f56e353dfdf44ec

github-actions[bot] avatar Oct 14 '22 13:10 github-actions[bot]

Newest code from mattermod has been published to preview environment for Git SHA 5a4fc4faa7936e7154485b123f56e353dfdf44ec

github-actions[bot] avatar Oct 14 '22 13:10 github-actions[bot]

Thanks for your contribution, @Drishti-jain21! Please make sure you have signed the CLA. :slightly_smiling_face:

neflyte avatar Oct 17 '22 14:10 neflyte

/check-CLA

Drishti-jain21 avatar Oct 17 '22 14:10 Drishti-jain21

Newest code from cwarnermm has been published to preview environment for Git SHA 5a4fc4faa7936e7154485b123f56e353dfdf44ec

github-actions[bot] avatar Oct 21 '22 19:10 github-actions[bot]

Newest code from cwarnermm has been published to preview environment for Git SHA 5a4fc4faa7936e7154485b123f56e353dfdf44ec

github-actions[bot] avatar Oct 21 '22 19:10 github-actions[bot]

@Drishti-jain21 Could you please include the source to the diagrams so they can be updated in the future should the guidelines change?

Also, you may wish to consider using a MermaidJS diagram which is supported in these docs.

neflyte avatar Oct 24 '22 16:10 neflyte

The diagrams are made upon the content from this page https://developers.mattermost.com/contribute/why-contribute/#youve-found-a-bug

I am not familiar with MermaidJs and I'll have to redo them again

Drishti-jain21 avatar Oct 24 '22 17:10 Drishti-jain21

/check-cla

azigler avatar Oct 27 '22 22:10 azigler

@azigler We're going to publish these great image assets to the developer site as-is knowing that we don't have an easy way to maintain them over time. @neflyte will add a card to the dev docs board to convert the existing image files to Mermaid for ease of maintainability in a future release.

cwarnermm avatar Oct 31 '22 14:10 cwarnermm

Newest code from cwarnermm has been published to preview environment for Git SHA 4d8474978069fc0c803fe69d4482ca7a3d224d32

github-actions[bot] avatar Oct 31 '22 14:10 github-actions[bot]

@Drishti-jain21 - We will move forward with your PR and merge it into our codebase once you've signed the CLA.

cwarnermm avatar Nov 02 '22 19:11 cwarnermm

I have signed the CLA

Drishti-jain21 avatar Nov 03 '22 11:11 Drishti-jain21

Please try again, @Drishti-jain21.I believe you can check your CLA status by entering /check cla as a comment to confirm.

cwarnermm avatar Nov 04 '22 12:11 cwarnermm

/check cla

Drishti-jain21 avatar Nov 04 '22 14:11 Drishti-jain21

/check-cla

Drishti-jain21 avatar Nov 04 '22 14:11 Drishti-jain21