Skip to content

Conversation

@SharkyBytes
Copy link
Contributor

@SharkyBytes SharkyBytes commented Mar 17, 2025

📝 Description

This pull request significantly improves our analytics dashboard by integrating real-time GitHub repository data fetched through our FastAPI backend. What I did here,

->Implemented repository input page: Added a dedicated front page for DevrAI, allowing users to input a GitHub repository URL for analysis. (UI improvements can be considered for a better user experience.)

image

-> Replaced hardcoded statistics with live GitHub data: Integrated real-time metrics, including contributors, pull requests, issues, and code activity, providing dynamic insights into repository health.

Contributors Section

image

Dashboard (Updated the Open PRs and Active contrbutors section to take the data dynamically)

image

PullRequests : Fetched all the states of the PR with the author

image

->Updated charts and graphs to dynamically render based on fetched data

->Minimized API calls by centralizing data fetching and distributing it across components, improving overall application responsiveness.

->Enriched the JSON response with additional repository metrics, ensuring comprehensive analytics. (If anyone wants to refine the UI or improve the graphical representation, feel free to contribute.)

Updated Data ->
response_1742200242697.json

image

image

-> Merged the latest codes from the frontend and backend repositories, which we were not having from the previous PRs.

-> Adjusted the environment configuration file to align with the current backend architecture.

@chandansgowda , Please take a look :)

✅ Checklist

  • I have read the contributing guidelines.
  • I have added tests that prove my fix is effective or that my feature works.
  • I have added necessary documentation (if applicable).
  • Any dependent changes have been merged and published in downstream modules.

@smokeyScraper
Copy link
Contributor

smokeyScraper commented Apr 12, 2025

@SharkyBytes, can you please address the conflicts? might have been introduced after merging of #58

@SharkyBytes
Copy link
Contributor Author

Hey @chandansgowda @kartikbhtt7, I’ve resolved all the conflicts in this PR caused by the recent code merges. Could you please review and merge it now?

@smokeyScraper
Copy link
Contributor

looks good to go. @chandansgowda can you please merge.
Thanks @SharkyBytes !!!

@chandansgowda chandansgowda merged commit ed332b7 into AOSSIE-Org:main Apr 13, 2025
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.

3 participants