Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Updating chart data with MakeUofT 2023 and NewHacks 2023 #514

Merged
merged 3 commits into from
Jan 29, 2024
Merged

Updating chart data with MakeUofT 2023 and NewHacks 2023 #514

merged 3 commits into from
Jan 29, 2024

Conversation

jhimanish
Copy link
Contributor

@jhimanish jhimanish commented Jan 29, 2024 •
edited
Loading

Overview

  • Resolves #184
  • Adding the MakeUofT 2023 and NewHacks 2023 application data to the charts on the admin dashboard
image image

Unit Tests Created

  • None because these are changes to the admin site

Steps to QA

@jhimanish jhimanish added the Admin Site Something on the admin site label Jan 29, 2024
Mustaballer
Mustaballer approved these changes Jan 29, 2024
View reviewed changes
Copy link
Contributor

@Mustaballer Mustaballer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work! After this is merged can you pls copy your changes over to the makeuoft 2024 repo :)

@Mustaballer Mustaballer merged commit 1dc1d95 into develop Jan 29, 2024
3 checks passed
@Mustaballer Mustaballer deleted the feat/update_data_charts branch January 29, 2024 04:09
@jhimanish jhimanish mentioned this pull request Jan 30, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LunarFang416 LunarFang416 LunarFang416 approved these changes

@Mustaballer Mustaballer Mustaballer approved these changes

Assignees
No one assigned
Labels
Admin Site Something on the admin site
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jhimanish @Mustaballer @LunarFang416