teammates icon indicating copy to clipboard operation
teammates copied to clipboard

Instructor: allow renaming of teams without losing existing submissions

Open damithc opened this issue 10 years ago • 7 comments

Currently, editing team name of a student is treated as 'one student moving to another team' which results in past submissions related to the previous team being deleted. What we need is a way to rename the whole team.

If the new team name matches an existing team, we should merge the two teams.

damithc avatar May 02 '14 13:05 damithc

1767-image-1

@damithc is this an expected UI? I feel that it should be more intuitive to add the checkbox here since this is the only place for instructor to edit student's team name. We can also add a tooltip for it (i.e. If checked, team name of all the students inside the team will be updated).

dalessr avatar Jul 27 '18 07:07 dalessr

Still, it's a bit odd to find that feature tucked in here. Can we have a separate Rename team button somewhere? When clicked, it will present a modal with a drop down to selected which team to rename and a text box to type the new name.

damithc avatar Jul 27 '18 07:07 damithc

In InstructorStudentListPage there's a panel for student list of each course. Shall we make use of the panel heading?

1767-image-2

dalessr avatar Jul 27 '18 08:07 dalessr

Sounds good. We can replicate it in other relevant places later.

damithc avatar Jul 27 '18 08:07 damithc

Yeah. In InstructorHomePage can also add an option under Students button:

1767-image-3

dalessr avatar Jul 27 '18 08:07 dalessr

It seems this issue has been resolved under the "instructors: can 'rename' of teams without losing existing submissions #1767 #5500" closed issue. Should this issue still be open? I apologize if I have said this in error.

JustinSingleton303 avatar Sep 22 '20 03:09 JustinSingleton303

It seems this issue has been resolved under the "instructors: can 'rename' of teams without losing existing submissions #1767 #5500" closed issue. Should this issue still be open? I apologize if I have said this in error.

You'll notice that PR #5500 was closed without merging. If you'd like to work on this issue, you can take a look at that PR and learn from what's already been done.

madanalogy avatar Sep 22 '20 04:09 madanalogy