Instructor permission test should also test correct data visibility #526
Assignees
Labels
test
Testing code
Comments
|
@siefkenj is there more context for this one? What data visibility are we talking about in the instructor permission tests? |
This was
linked to
pull requests
Jul 5, 2021
|
Phone number should not be visible to instructors. Currently instructor is a restricted admin, i.e. they have the same init procedure, but get a subset of existing data. We need to go through some fields and see if they should be seen by instructors. After that we should write tests for these fields to be restricted. |
|
FAS is worried about sessional instructors having permissions to see grad students, etc. Instructors should NOT see applications and see applicants.As a first iteration at least. We might want to let them do that still - TBD |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
PR #477 is getting bloated; therefore this section will not be implemented in this PR. After this PR is merged a new PR will be created to implement tests for data visibility.
The text was updated successfully, but these errors were encountered: