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

Add multiple teachers to a classroom #166

Open
GuillermoFloresV opened this issue Oct 26, 2022 · 6 comments
Open

Add multiple teachers to a classroom #166

GuillermoFloresV opened this issue Oct 26, 2022 · 6 comments
Assignees

Comments

@GuillermoFloresV
Copy link
Member

GuillermoFloresV commented Oct 26, 2022

Describe the bug

We only support one teacher/instructor per class, however, this may not fit all use cases. We would like to be able to have multiple teachers (limit yet to be discussed). The first step would require us to change the Prisma schema / database to support multiple teachers for a given classroom.

Here is how you can approach this task:

  1. Open schema.prisma
  2. Update the Classroom model ==> classroomTeacherId should take an array of Strings. Afterwards you will need to do a prisma migration or wipe the DB and start from scratch.
  3. Search the codebase for all references to classroomTeacherId in javascript files. Update the code so that works with an array. For example, you may want to append to the classroomTeacherId array instead of setting it directly.
@diegovega49
Copy link

Hello @GuillermoFloresV I would like to be assigned to this

@jtnguyen45
Copy link

My team and I from CTI are working on this.

@utsab
Copy link
Collaborator

utsab commented Feb 5, 2023

@jtnguyen45 I updated the original description of this issue to reduce the scope of the issue and make it more clear.

@jtnguyen45
Copy link

Sounds great @utsab . Thank you for the clarification.

@theGaryLarson
Copy link
Contributor

Hi @GuillermoFloresV it looks like people have said they are working on this but it looks dated and no one is currently assigned. I would like to take this on if possible?

@utsab
Copy link
Collaborator

utsab commented Jul 18, 2023

HI @theGaryLarson -- I assigned the issue to you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

No branches or pull requests

5 participants