mentorship-backend icon indicating copy to clipboard operation
mentorship-backend copied to clipboard

Updating an user with empty username and name should give an error message

Open faznan3nazer opened this issue 4 years ago • 22 comments

When updating an user with empty username and name should give an error message.

To Reproduce

Steps to reproduce the behavior:

  1. Go to 'User-update user profile'.

  2. Provide authorization access token.

  3. { "name": "", "username": "", "bio": "string", "location": "string", "occupation": "string", "organization": "string", "slack_username": "string", "social_media_links": "string", "skills": "string", "interests": "string", "resume_url": "string", "photo_url": "string", "need_mentoring": true, "available_to_mentor": true }

  4. Click on 'Execute'.

  5. Scroll down to 'Responses'.

Screenshot (75) 7. current user's detail after update in" Return details of current user". Screenshot (76)

Expected behavior If the username and name are not expected to be updated to empty string and error should be shown.

Desktop (please complete the following information):

  • OS: Windows 10
  • Browser chrome

faznan3nazer avatar May 17 '20 20:05 faznan3nazer

Hi is this bug still open. I want to work on this.

rashikaqureshi avatar Jun 20 '20 11:06 rashikaqureshi

I would like to work on this issue.

Sheetalsingh23 avatar Jun 22 '20 12:06 Sheetalsingh23

@rashikaqureshi sorry for the late reply do you still want to work on this?

isabelcosta avatar Jul 19 '20 18:07 isabelcosta

@isabelcosta yes.

rashikaqureshi avatar Jul 20 '20 08:07 rashikaqureshi

yey! I will assign this to you then @rashikaqureshi :)

isabelcosta avatar Jul 20 '20 18:07 isabelcosta

@isabelcosta @devkapilbansal I don't see any updates on the PR by the current assignee so, can I work on this?

Aaishpra avatar Dec 22 '20 14:12 Aaishpra

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

devkapilbansal avatar Dec 22 '20 16:12 devkapilbansal

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

No problem, The pr is already 5 months old that's why i asked.

@rashikaqureshi can you solve the conflicts.

Aaishpra avatar Dec 22 '20 16:12 Aaishpra

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

devkapilbansal avatar Dec 22 '20 16:12 devkapilbansal

@Aaishpra please tag the team on zulip and asked for this issue. This issue will be assigned to you then. Cc:- @vj-codes

devkapilbansal avatar Dec 31 '20 19:12 devkapilbansal

@Aaishpra hey I will be closing the linked PR due to inactivity and the issue will be available again , however it's a first timers only issue and unfortunately you can't be assigned to this . Do comment on any other issue you would like to work on:) cc: @devkapilbansal

vj-codes avatar Dec 31 '20 20:12 vj-codes

@Aaishpra hey I will be closing the linked PR due to inactivity and the issue will be available again , however it's a first timers only issue and unfortunately you can't be assigned to this . Do comment on any other issue you would like to work on:) cc: @devkapilbansal

Sure, No problem

Aaishpra avatar Jan 01 '21 13:01 Aaishpra

I would like to work on this issue.

surbhigoel191 avatar Jan 04 '21 18:01 surbhigoel191

HI @isabelcosta , @vj-codes :wave:. I would like to take on this?

NenadPantelic avatar Jan 08 '21 11:01 NenadPantelic

@surbhigoel191 @NenadPantelic this is a first timers only issue and both of you have already made your first contributions so unfortunately this can't be assigned to you . Do go through other issues and comment :)

vj-codes avatar Jan 08 '21 12:01 vj-codes

Hi @isabelcosta @vj-codes I would like to work on this issue. Can this issue be assigned to me?

battuAshita avatar Jan 15 '21 11:01 battuAshita

@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

devkapilbansal avatar Jan 16 '21 22:01 devkapilbansal

@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

Ok:)

battuAshita avatar Jan 17 '21 09:01 battuAshita

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

devkapilbansal avatar Mar 20 '21 08:03 devkapilbansal

I mentioned this issue under a different issue, but I would like to be assigned this issue instead of #673 if possible!

mariejp avatar Apr 14 '22 23:04 mariejp

As @devkapilbansal mentioned it might have been resolved earlier so if you could first check if this is still reproducible that would be great : ).

epicadk avatar Apr 15 '22 06:04 epicadk

Hi! I did notice that the Pull Request had already gone through for this issue so I instead began working on #685 ! I created a pull request here

mariejp avatar Apr 23 '22 02:04 mariejp