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

[MM-468]: Include number of lines changed in RHS PR list #41

Closed
wants to merge 4 commits into from

Conversation

Kshitij-Katiyar
Copy link

Summary

Include the number of lines changed in the RHS PR list

Screenshot

Existing

Screenshot from 2024-09-03 16-55-42
Screenshot from 2024-09-03 16-55-52

Updated

Screenshot from 2024-09-03 16-50-28
Screenshot from 2024-09-03 16-50-41

How to test

  • Connect your mattermost to GitHub
  • Create a PR on GitHub with another account and add the GitHub account connected with mattermost as the reviewer
  • Open the RHS PR list

server/plugin/graphql/lhs_request.go Show resolved Hide resolved
server/plugin/graphql/lhs_request.go Outdated Show resolved Hide resolved
webapp/src/components/sidebar_right/github_items.tsx Outdated Show resolved Hide resolved
@Kshitij-Katiyar
Copy link
Author

Created a PR on mattermost remote with all these changes hence closed this one

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants