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
Instructor: enroll students: allow same team name across sections #2618
Comments
@damithc this would change the following method teammates/src/main/java/teammates/logic/core/StudentsLogic.java Lines 423 to 449 in 30a7659
In that case, what would constitute an invalid team info? |
Nothing I guess. However, note that we need to tweak our UI (and the logic) in many place to distinguish multiple teams with the same name but in different sections. e.g., show |
Should a new variable be added in TeamDetailsBundle class, i.e. the sectional this team belongs to? Since this is a big change, I want to be clear of the steps taken. Here are some of the areas that would be affected by the change. Let me know if there are other considerations. Right now I have
How should I proceed to solve this issue systematically i.e. which part of the code base should be changed before the others? @wkurniawan07 would love to have your input on this as well! |
@Haozhe321 I'm not familiar with this feature myself, but if anything, the result should be the last of all the worries (i.e. worry about the |
'team 1' in tutorial 1 should be considered different from 'team 1' in tutorial 2
The text was updated successfully, but these errors were encountered: