dashboard icon indicating copy to clipboard operation
dashboard copied to clipboard

Feature: create stats dashboard page

Open chenxuan-zhou opened this issue 4 years ago • 2 comments

Tickets:

List of changes:

  • added admin stats page to display application counts
  • added application count of current day
  • added line graph of application count of past 30 days
  • merged in demographic stats from PR 961

Type of change:

Please delete options that aren't relevant.

  • [ ] New feature (non-breaking change which adds functionality)

How did you do this?

Created the Stats and StatsApplicationGraph components in features. Added a new "/admin/stats" route for the Stats component.

How to test:

Automatic test at PR.

Questions:

I did not push the changes done by linting since many code that I didn't touch were also changed and there were many errors. Do I need to worry about this?

I had an interface that I would like to share between Stats and StatsApplicationGraph. Is there a good way of doing so? (Currently I just copied the interface over)

PR Checklist:

  • [ ] Merged develop branch (before testing)
  • [ ] Linted my code locally
  • [ ] Listed change(s) in the Changelog
  • [ ] Tested all links in project relevant browsers
  • [ ] Tested all links on different screen sizes
  • [ ] Referenced all useful info (issues, tasks, etc)

Screenshots:

Search _ McHacks — Mozilla Firefox 11_29_2021 4_07_36 PM Search _ McHacks — Mozilla Firefox 11_29_2021 4_07_41 PM

chenxuan-zhou avatar Nov 16 '21 22:11 chenxuan-zhou

You can add & export an interface as a separate file and then import it into both files you want to use it. We do this for other interfaces too!

pierreTklein avatar Nov 19 '21 00:11 pierreTklein

You can add & export an interface as a separate file and then import it into both files you want to use it. We do this for other interfaces too!

Thank you Theo! I put the interface in a separate file in the latest commit. Does the export style look correct?

chenxuan-zhou avatar Nov 25 '21 22:11 chenxuan-zhou