Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#167190438 User's Profile #17

Merged
merged 10 commits into from
Aug 1, 2019
Merged

Conversation

Hadeneekeh
Copy link
Contributor

What does this PR do?

  • It has GET /api/user which creates the user's profile
  • It has PUT /api/user which edits the user's profile
  • It has GET /api/profiles/:username which has viewing another user's profile

Description of Task to be completed?

  • write test
  • Created the following endpoints:
    • get authorised user's profile
    • edit authorised user's profile
    • view any user's profile
  • document the feature on swagger

How should this be manually tested?

  • fetch or pull ft/167190438/create-user-profile
  • run npm install to get the packages used
  • Test the endpoint above using postman as demonstrated in the screenshots below

Any background context you want to provide?

  • use the string 'image' as the key for the image upload.

What are the relevant pivotal tracker stories?

#167190438

Screenshots (if appropriate)

image

Questions:

NA

Copy link
Contributor

@cvjude cvjude left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@encodedBicoding encodedBicoding left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@chiboycalix chiboycalix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@chiboycalix chiboycalix force-pushed the ft/167190438/create-user-profile branch from 95bbe1d to 0e406d7 Compare August 1, 2019 13:58
Get user's profille

Updates #167190438
get any user's profile

[Finishes #167190438]
improve test

[Updates #167190438]
edit user's profile

[Finishes #167190438]
edit user's profile

[Finishes #167190438]
fix merge conflict

[Delivers #167190438]
fix merge conflict

[Delivers #167190438]
Improve implementation and add validation

[Delivers #167190438]
Improve implementation and add validation

[Delivers #167190438]
@Hadeneekeh Hadeneekeh force-pushed the ft/167190438/create-user-profile branch from 0e406d7 to 4a5df33 Compare August 1, 2019 14:32
Fix failing test

[Delivers #167190438]
@topseySuave
Copy link
Contributor

Please next time try to reduce the amount similar commits you make

@topseySuave topseySuave merged commit 54d9088 into develop Aug 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants