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

UI confusion between "Student Files" and "Student Lookup" #46

Open
jamesjuett opened this issue Aug 15, 2023 · 1 comment
Open

UI confusion between "Student Files" and "Student Lookup" #46

jamesjuett opened this issue Aug 15, 2023 · 1 comment

Comments

@jamesjuett
Copy link

jamesjuett commented Aug 15, 2023

It appears it is too easy to mistake the "Student Files" section in project configuration for "Student Lookup".

When someone accidentally tries to look up a student in "Student Files", their uniqname gets added as a required file.

image

Of course, this interferes with students submitting the project. At the same time, the poor student who someone was trying to look up has their uniqname plastered on Piazza as people post the error message they're getting.

The interface is clear to me, but perhaps only because I have the advantage of extensive experience configuring the autograder. I suggest a change to the UI to prevent this. Two ideas:

  1. Add a bold, ugly warning to the UI that "THIS IS NOT STUDENT LOOKUP" when you're on the "Student Files" config tab.
  2. If a project is currently published, the frontend UI makes changing student files more involved. For example, it could disable the "Add File" button unless you check a box that says something like "yes I know what I'm doing and I really want to add a new file that all students must submit".

1 is less work, but 2 is probably a better solution.

@james-perretta
Copy link
Contributor

What about changing the header to "Expected Student Files"?

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

No branches or pull requests

2 participants