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

Update README.md #191

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

Conversation

Aatmaj-Zephyr
Copy link

Summary

Adds Screenshot to README.md file

Ticket Link

Fixes #121

@mattermod
Copy link
Contributor

Hello @Aatmaj-Zephyr,

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
Copy link
Contributor

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

Thanks, @Aatmaj-Zephyr! This is a great PR. Please sign the CLA so we can go ahead and merge this once we have the second review.

@larkox
Copy link
Contributor

larkox commented Nov 24, 2022

/update-branch

@larkox
Copy link
Contributor

larkox commented Nov 24, 2022

/check-cla

@larkox
Copy link
Contributor

larkox commented Nov 24, 2022

@Aatmaj-Zephyr have you signed the CLA? See the comment here for more information: #191 (comment)

@Aatmaj-Zephyr
Copy link
Author

@Aatmaj-Zephyr have you signed the CLA? See the comment here for more information: #191 (comment)

I am getting this issue when I click on the link

image

@larkox
Copy link
Contributor

larkox commented Nov 28, 2022

@Aatmaj-Zephyr Try this link

@cwarnermm cwarnermm added the Awaiting Submitter Action Blocked on the author label Nov 28, 2022
@codecov-commenter
Copy link

Codecov Report

Base: 7.56% // Head: 7.56% // No change to project coverage 👍

Coverage data is based on head (7fa8d77) compared to base (ef0aa21).
Patch has no changes to coverable lines.

Additional details and impacted files
@@          Coverage Diff           @@
##           master    #191   +/-   ##
======================================
  Coverage    7.56%   7.56%           
======================================
  Files          10      10           
  Lines        1282    1282           
======================================
  Hits           97      97           
  Misses       1177    1177           
  Partials        8       8           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@mattermod
Copy link
Contributor

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!

@larkox
Copy link
Contributor

larkox commented Dec 12, 2022

@Aatmaj-Zephyr Where you able to sign the CLA?

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.

Add screenshot to the README
6 participants