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

late days allowed 'removing' entries #1848

Closed
bmcutler opened this issue Apr 6, 2018 · 1 comment
Closed

late days allowed 'removing' entries #1848

bmcutler opened this issue Apr 6, 2018 · 1 comment

Comments

@bmcutler
Copy link
Member

bmcutler commented Apr 6, 2018

Bug:

The "late days allowed" tab from the instructor user interface does not allow us to remove an entry in the late_days table added for a student. If we have made an accidental entry for a student, we should be able to remove that. The workaround now is to re-specify the default for that student.

The page has an attempt/hack to do so: If you re-enter the same date and specify "Late Days" as 0, then the row disappears from the tab display, but it's still in the database table (with the value 0), and queries that use the table, will have zero, which is wrong, if the course has a non-zero default number of late days.

Instead, this tab display should show ALL rows in the table (sorted by username, then date), and we should have a pencil or x icon to delete a row. And we should be able to specify '0' late days for a student -- to zero out the default number allowed.

@scopeInfinity
Copy link
Contributor

Hi, @bmcutler
I just wanted to know if apart from adding 'x' icon for deletion of a row, is there any requirement for Multiple Entry Deletion Via CSV Upload?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants