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

On People page, show Display Name for students' view and Full Name and d Display Name for teachers and TA's view #250

Closed
wants to merge 1 commit into from

Conversation

GinnyGuo
Copy link

The reason we do this is that instructors would like to view all the users with both legal full name and display name participating in the course on People page, while many students would like their classmates to see their Display Name on People page rather than legal Full Name to avoid embarrassment and to be consistent with their names in discussions, announcements, etc.

Test plan:

  • Go to a course
  • Open the People page
  • As a teacher, verify both Full Name and Display Name showing in Name column
  • As a TA, verify both Full Name and Display Name showing in Name column
  • As a student, verify just Display Name showing in Name column

…d Display Name for teachers and TA's view

The reason we do this is that instructors would like to view all the users with both legal full name and display name participating in the course on People page, while many students would like their classmates to see their Display Name on People page rather than legal Full Name to avoid embarrassment and to be consistent with their names in discussions, announcements, etc.

Test plan:

* Go to a course
* Open the People page
* As a teacher, verify both Full Name and Display Name showing in Name column
* As a TA, verify both Full Name and Display Name showing in Name column
* As a student, verify just Display Name showing in Name column
@GinnyGuo GinnyGuo closed this Apr 19, 2016
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

Successfully merging this pull request may close these issues.

1 participant