Skip to content
This repository has been archived by the owner on Jul 29, 2023. It is now read-only.

[WIP] Fix the miscounting issue #57

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

[WIP] Fix the miscounting issue #57

wants to merge 1 commit into from

Conversation

shubhsherl
Copy link
Owner

Closes #49 #50 #56

@shubhsherl
Copy link
Owner Author

shubhsherl commented Apr 21, 2019 •
edited
Loading

Changes in refresh workflow to fix the major bugs:

  • After Initial fetching, Relation model stores the issues/Prs of user for a repo, with lastMerged(contains merged_at), lastIssue(contains created_at), lastClosed(contains closed_at), lastOpenPr(contains created_at).
    Initially, only created_at was stored and used for refreshing data.

@shubhsherl shubhsherl mentioned this pull request Apr 22, 2019
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[BUG] Leaderboard shows wrong open PR count
1 participant
@shubhsherl