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

Feature/116/GitHub information #121

Draft
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

martinrobled0
Copy link
Contributor

What does this PR do?

Where should the reviewer start?

How should this be manually tested?

Any background context you want to provide?

What are the relevant tickets?

Screenshots

Questions

Checklist

  • I added the necessary documentation, if appropriate.
  • I added tests to prove that my fix is effective or my feature works.
  • I reviewed existing Pull Requests before submitting mine.

@jackbravo
Copy link
Collaborator

These are some good first steps in the integration. Showing that we can connect and bring information like commits. But I think we need to also think about how we are going to store this information in our database since we don't want to be querying GH every time we show this info.

Also, I don't think we need to show a table with commits, but instead, try to show some metrics of whether the project is advancing or not, since that is the main question we want to answer.

Are projects progressing?
And also, who is participating in this progress?

For these two questions, I think GitHub can help us with:

  • commits made, by the project and by the individual
  • issues opened and closed (this can be made by non-developers too)
  • PRs opened and closed
  • releases made

And we should think on how we want to display this data. Maybe both on the project detail page and also on the project search.

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