Contribution dashboard admin change: Add overall item count to contributor dashboard admin stats table - Githubissues

oppia / oppia

A free, online learning platform to make quality education accessible for all.

https://www.oppia.org

Apache License 2.0

5.63k stars 3.78k forks source link

Contribution dashboard admin change: Add overall item count to contributor dashboard admin stats table #20186

Closed stevenjgodfrey closed 18 hours ago

stevenjgodfrey commented 3 weeks ago

Overview

Add overall item count to contributor dashboard admin stats table

This PR fixes or fixes part of #19570 .
This PR does the following: Add overall item count to contributor dashboard admin stats table

Essential Checklist

Please follow the instructions for making a code change.

[x] I have linked the issue that this PR fixes in the "Development" section of the sidebar.
[x] I have checked the "Files Changed" tab and confirmed that the changes are what I want to make.
[x] I have written tests for my code.
[x] The PR title starts with "Fix #bugnum: " or "Fix part of #bugnum: ...", followed by a short, clear summary of the changes.
[x] I have assigned the correct reviewers to this PR (or will leave a comment with the phrase "@{{reviewer_username}} PTAL" if I can't assign them directly).

Proof that changes are correct

https://github.com/oppia/oppia/assets/123914818/00553b0e-6127-4907-a774-f0ee5f1496eb

oppiabot[bot] commented 3 weeks ago

Assigning @Nik-09, @StephenYu2018 for the first pass review of this PR. Thanks!

stevenjgodfrey commented 3 weeks ago

@seanlip PTAL @chris7716 PTAL

stevenjgodfrey commented 2 weeks ago

@seanlip PTAL, @StephenYu2018 PTAL

oppiabot[bot] commented 2 weeks ago

Unassigning @stevenjgodfrey since a re-review was requested. @stevenjgodfrey, please make sure you have addressed all review comments. Thanks!

oppiabot[bot] commented 2 weeks ago

Unassigning @StephenYu2018 since the review is done.

oppiabot[bot] commented 2 weeks ago

Hi @stevenjgodfrey, it looks like some changes were requested on this pull request by @StephenYu2018. PTAL. Thanks!

stevenjgodfrey commented 2 weeks ago

@StephenYu2018 PTAL

oppiabot[bot] commented 2 weeks ago

Unassigning @stevenjgodfrey since a re-review was requested. @stevenjgodfrey, please make sure you have addressed all review comments. Thanks!

oppiabot[bot] commented 2 weeks ago

Unassigning @StephenYu2018 since the review is done.

oppiabot[bot] commented 2 weeks ago

Hi @stevenjgodfrey, it looks like some changes were requested on this pull request by @StephenYu2018. PTAL. Thanks!

stevenjgodfrey commented 2 weeks ago

@StephenYu2018 PTAL

oppiabot[bot] commented 2 weeks ago

Unassigning @stevenjgodfrey since a re-review was requested. @stevenjgodfrey, please make sure you have addressed all review comments. Thanks!

oppiabot[bot] commented 2 weeks ago

Unassigning @StephenYu2018 since the review is done.

oppiabot[bot] commented 2 weeks ago

Hi @stevenjgodfrey, it looks like some changes were requested on this pull request by @StephenYu2018. PTAL. Thanks!

oppiabot[bot] commented 1 week ago

Hi @stevenjgodfrey, I'm going to mark this PR as stale because it hasn't had any updates for 7 days. If no further activity occurs within 4 days, it will be automatically closed so that others can take up the issue. If you are still working on this PR, please make a follow-up commit within 4 days (and submit it for review, if applicable). Please also let us know if you are stuck so we can help you!

stevenjgodfrey commented 1 week ago

@StephenYu2018 PTAL

oppiabot[bot] commented 1 week ago

Unassigning @stevenjgodfrey since a re-review was requested. @stevenjgodfrey, please make sure you have addressed all review comments. Thanks!

oppiabot[bot] commented 1 week ago

Unassigning @StephenYu2018 since they have already approved the PR.

stevenjgodfrey commented 6 days ago

@Nik-09 PTAL

oppiabot[bot] commented 6 days ago

Unassigning @chris7716 since they have already approved the PR.

chris7716 commented 19 hours ago

@Nik-09 PTAL

oppiabot[bot] commented 19 hours ago

Hi @stevenjgodfrey, this PR is ready to be merged. Please address any remaining comments prior to merging, and feel free to ask someone to merge your PR once the CI checks pass and you're happy with it. Thanks!