Closed vuvanhieu143 closed 1 year ago
Oops! I clicked merge before acutally reviewing the code! Fortunately, it is good.
I did, however, improve some coment (commit 6fbb1ba858ee13634fca0f882a3a81907bd42220) based on the things I had to work out while reviewing.
Hi @timhunt ,
I have fixed the issue to make sure we only send to the tutors in the group that students belong to. I add some unit tests as well.
Note: the unit test failed is not related to this issue, It will be fixed in ticket 683232