Delete Student Record Button Is Broken

[cycomachead]

On the table of students for an exam, trying to delete a student leads to a 404 because the record id seems to be missing from the URL

https://seating-staging-02a3510a1d0c.herokuapp.com/offerings/1528451/exams/final/students//delete

[Reimirno]

We need more info on this. I cant seem to reproduce it.

We create student rows when the user import student roster (from canvas or google sheet).
And then in this deletion endpoint we will search for that student to delete.
Url interpolation looks right, and it is also not possible that student has no canvas id (we have schema validation at both app layer and db layer).

I think we need to find a way to reproduce this error. Can we still access last sem's cs88 data?

[cycomachead]

I’ll add you to the fall bcourses later. I wonder if an exception was raised or not if this is the type of thing we could capture in sentry. -- Michael BallFrom my iPhonemichaelball.coOn Feb 7, 2024, at 10:38 AM, Yu Long ***@***.***> wrote: We need more info on this. I cant seem to reproduce it. image.png (view on web) We create student rows when the user import student roster (from canvas or google sheet). And then in this deletion endpoint we will search for that student to delete. Url interpolation looks right, and it is also not possible that student has no canvas id (we have schema validation at both app layer and db layer). I think we need to find a way to reproduce this error. Can we still access last sem's cs88 data? —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: ***@***.***>