WYHNUS / ExchangeBuddy

Find your exchange buddies!
5 stars 0 forks source link

/patch updateUser endpoint bug #107

Closed kaiyisg closed 7 years ago

kaiyisg commented 7 years ago

@ZhangHanming when i send the request, does not update the server.

also, disable updating email as that is not desired behavior, and enable updating password i will only need to update these fields:

if can integrate image uploading at this endpoint and tell me how, would be good too

@WYHNUS is this the same problem as what sam said, should not update by passing userId, if not anyone else can update for u? if so, what should i do to identify users instead?

{ 'userId': //to access who is the one updating 'name': 'password': }

WYHNUS commented 7 years ago

Email is the primary key which should not be updated. It will be desirable to enable link different email addresses together. (which we don't need to consider now...)

Identify through token stored in user reducer.

ZhangHanming commented 7 years ago

@kaiyisg update user endpoint fixed

kaiyisg commented 7 years ago

@ZhangHanming endpoint not fixed. refer to #87 number 3. still does not update the info when i tested