Evalai: Cannot remove teams from existing host team view

[anweshknayak]

[aka-jain]

Thanks @anweshknayak for pointing this out, actually we have some API problem. @deshraj please take care of it :P

[deshraj]

Sure. I am working on another issue as of now so might take time to fix this. Anyone wants to take this issue, then go ahead. :-)

[gauthamzz]

@deshraj could i try ?

[deshraj]

Sure.

[aka-jain]

@gauthamzz Progress?

[gauthamzz]

@aka-jain i'm not sure where the bug is . still finding out . i dont think its frontend . I need some more time

[aka-jain]

Yes @gauthamzz it is not from frontendside, actually API is not working properly. Please check accordingly.

[yadavankit]

633 @deshraj please review this PR for this issue too

[yadavankit]

@deshraj the code already works fine as it removes the user from ChallengeHost but the team shows up because the User is still the owner of the team. One more thing, in Host Challenge page, user should be able to view all teams he is a part of (as of now, one can only view teams created by him). What do you say?

[deshraj]

@deshraj the code already works fine as it removes the user from ChallengeHost but the team shows up because the User is still the owner of the team.

@yadavankit I think you should add a check where you check if the user is the owner of the Team and if he is the part of the team then delete the whole team since he is the owner of the team.

One more thing, in Host Challenge page, user should be able to view all teams he is a part of (as of now, one can only view teams created by him). What do you say?

Good catch. Can you please open a issue for the same?

[yadavankit]

@deshraj yeah sure 👍

[deshraj]

@yadavankit are you working on this or @gauthamzz is working?

[yadavankit]

@deshraj I did not start on this issue as it is assigned to @gauthamzz. Should I?

[gauthamzz]

@deshraj told me someone else is working on this , so i stopped working on this

[anweshknayak]

I just tested this. It is working fine. Still anyone can confirm tho

[gauthamzz]

yeah its working fine now

[yadavankit]

@deshraj @anweshknayak if fine, then lets close it for now?

[deshraj]

Thanks everyone for confirming. :)