Kapil Bansal

Results 112 comments of Kapil Bansal

Thanks @jasminjohal for this issue. Would you like to work on it?

@Aaishpra please wait for some more time for the assignee to respond. As the PR is complete and needs some minor merge conflicts only

I tag the assignee again. If the assignee doesn't respond within two-three days then this issue can be assigned to you

@Aaishpra please tag the team on [zulip](https://anitab-org.zulipchat.com/) and asked for this issue. This issue will be assigned to you then. Cc:- @vj-codes

@battuAshita this issue is for *First TImers Only` and since you have already made a PR. You can't work on this issue. Go through other issues

May be this issue was resolved earlier :thinking: Removing `Available` for now.

> Shall I merge all commit in a single commit as I haven't created a new branch ? No, you don't need to. Just keep this in mind from next...

@aakankshaagr there are some suggestions above. Once done, this PR may get completed. Look into them once you have some time.

@aakankshaagr make changes in this line https://github.com/anitab-org/mentorship-backend/blob/c154317b266ffb258f6a7b2ab5ea95de45c5de81/tests/users/test_dao_update_user.py#L30 Pass occupation and organization as None here. Then your tests will not fail