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.

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

Dashboard facelift [REFERENCE] #538

Open
wants to merge 67 commits into
base: master
Choose a base branch
from

Conversation

evankirkiles
Copy link
Contributor

Was cleaning up branches and decided to move my dashboard off of my master and into its own branch so I could keep the upstream master and forked master the same. This PR is essentially the same as #524, though there are some small updates:

  • Added "Bonused" column which tallies up all bonuses paid to each assignment.
  • Added functionality for auto-bonusing from a CSV in the dashboard, even for non-local experiments.

Feel free to nab features––I'm creating this PR to maintain the link between my own development and the master. Screenshots below:

screenshot1

screenshot2

screenshot3

screenshot4

screenshot5

@coveralls
Copy link

coveralls commented Jan 5, 2022

Coverage Status

Coverage: 56.468% (-3.8%) from 60.219% when pulling 2050636 on evankirkiles:beta into af00724 on NYUCCL:master.

@deargle
Copy link
Collaborator

deargle commented Jan 5, 2022

Thanks! Good to have this reference.

Status update, I mocked a lot of the dashboard's style layout already, on this repo's master branch. But I'll not take the js though since I'm selfish and want to keep this repo on Vue.js. I probably will also keep this repo's api code the same as it is now.

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.

None yet

3 participants