Skip to content

To show Commits in Scrum Report #114

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

Merged
merged 48 commits into from
Jun 16, 2025
Merged

Conversation

Preeti9764
Copy link
Contributor

@Preeti9764 Preeti9764 commented Jun 11, 2025

📌 Added feature to show commits

Fixes #51


📝 Summary of Changes

The Commits made in existing PRs will be visible in generated Scrum along with the base PR.


📸 Screenshots / Demo (if UI-related)

image


✅ Checklist

  • I’ve tested my changes locally
  • I’ve added tests (if applicable)
  • My code follows the project’s code style guidelines

Summary by Sourcery

Add commit listing and caching enhancements to the Scrum report extension

New Features:

  • Include user commit details (SHA, message, link, PR status) in the generated Scrum report
  • Implement parallel GitHub API requests with a 10-minute TTL cache
  • Add a manual “Refresh Data” button to bypass cache and reload GitHub data

Enhancements:

  • Refactor data fetching to use modern fetch, Promises, and debug logging
  • Introduce a storage-backed caching layer with queue and error handling
  • Add UI buttons to rewrite Scrum in email composer and refresh cache in popup

Documentation:

  • Update README to document caching behavior, parallel fetches, and manual refresh

vedansh-5 added 30 commits May 28, 2025 23:13
Signed-off-by: Vedansh Saini <[email protected]>
Signed-off-by: Vedansh Saini <[email protected]>
Signed-off-by: Vedansh Saini <[email protected]>
Signed-off-by: Vedansh Saini <[email protected]>
Signed-off-by: Vedansh Saini <[email protected]>
Signed-off-by: Vedansh Saini <[email protected]>
Signed-off-by: Vedansh Saini <[email protected]>
Signed-off-by: Vedansh Saini <[email protected]>
Signed-off-by: Vedansh Saini <[email protected]>
Signed-off-by: Vedansh Saini <[email protected]>
Signed-off-by: Vedansh Saini <[email protected]>
Signed-off-by: Vedansh Saini <[email protected]>
Signed-off-by: Vedansh Saini <[email protected]>
Signed-off-by: Vedansh Saini <[email protected]>
Signed-off-by: Vedansh Saini <[email protected]>
Signed-off-by: Vedansh Saini <[email protected]>
Signed-off-by: Vedansh Saini <[email protected]>
vedansh-5 and others added 13 commits June 5, 2025 10:39
Signed-off-by: Vedansh Saini <[email protected]>
Signed-off-by: Vedansh Saini <[email protected]>
Signed-off-by: Vedansh Saini <[email protected]>
Signed-off-by: Vedansh Saini <[email protected]>
Signed-off-by: Vedansh Saini <[email protected]>
Signed-off-by: Vedansh Saini <[email protected]>
Signed-off-by: Vedansh Saini <[email protected]>
Signed-off-by: Vedansh Saini <[email protected]>
Signed-off-by: Vedansh Saini <[email protected]>
@Preeti9764
Copy link
Contributor Author

Preeti9764 commented Jun 11, 2025

Hello @hpdang @vedansh-5 Please check this once. I have used PR #94 as a base so that caching can be implemented and API calls limits do not exceed.Thanks!

@Preeti9764
Copy link
Contributor Author

Preeti9764 commented Jun 12, 2025

@hpdang @vedansh-5 I made changes to limit the api calls . Check out this . Thanks!

@vedansh-5
Copy link
Contributor

This new pushed code works fine, good work @Preeti9764 !

@hpdang hpdang merged commit bd182b9 into fossasia:master Jun 16, 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.

To show new commits and comments on a existing PR
3 participants