-
Notifications
You must be signed in to change notification settings - Fork 0
Grades #169
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
base: dev
Are you sure you want to change the base?
Grades #169
Conversation
|
@brandnholl is attempting to deploy a commit to the Coursify Studios Team on Vercel. To accomplish this, @brandnholl needs to request access to the Team. Afterwards, an owner of the Team is required to accept their membership request. If you're already a member of the respective Vercel Team, make sure that your Personal Vercel Account is connected to your GitHub account. |
|
The latest updates on your projects. Learn more about Vercel for Git βοΈ
|
|
This merge is awaiting the actual setup of grading and interactivity by @IllGive . Ideally we don't want it to fall too behind from dev though. |
sounds good |
Note that it might not work. I have not tested it yet. Not that I can, there are exactly zero graded submissions right now, and that IS my problem, but I'm too lazy to fix that right now. I must shower, so see you after then.
something is borked. also disabled/changed some stuff because of changes to db types
sorry about this, but if we want this implemented, someone is going to have to take over from me for the next 2 weeks |
| referencedColumns: ["test"]; | ||
| }, | ||
| ]; | ||
| }; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
iirc these tables have since been deleted
| return await Promise.all([ | ||
| supabase | ||
| .from("submissions") | ||
| //no need to fetch user_id when we have rls |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
uhhhh
| setGrade( | ||
| data.data.class_users.find((v) => v.user_id == user.id)?.grade | ||
| //data.data.class_users.find((v) => v.user_id == user.id)?.grade | ||
| 100 //TODO: fix |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix this
| <div className="w-full"> | ||
| <div className="w-full rounded-lg bg-gray-200 p-4 flex items-center justify-between"> | ||
| <div> | ||
| <h1 className="font-bold text-4xl">Honors Biology</h1> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably shouldn't be hardcoding classes
|
also has merge conflicts needing to be resolved |
π€ Generated by Copilot at 926c229
Summary
ππ οΈπ
This pull request adds a new feature to the app that lets the user view and edit their grades. It creates new components, pages, and routes for the grades page, the grade details page, and the grade settings section. It also updates a dependency and fixes a formatting issue.
Walkthrough
sharppackage (link)components/settings/index.tsxfile (link)