Kapil Bansal
Kapil Bansal
Thanks @jasminjohal for this issue. Would you like to work on it?
@RiddhiAthreya any updates?
@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