Skip to content

Conversation

@ananyar807
Copy link

ℹ️ Issue

Closes #2

📝 Description

Create an endpoint getUser that gets the user information for a current user.

  1. Added parameters to the user type
  2. created getUser method in controller
  3. created findOne method in service

✔️ Verification

@chromium-52 chromium-52 changed the base branch from main to ojn03/feat-getAllMembers October 4, 2023 01:30
@chromium-52 chromium-52 changed the title Ar/get user Add getUser endpoint Oct 4, 2023
@chromium-52 chromium-52 requested review from kimharr24 and ojn03 October 4, 2023 01:31
@chromium-52 chromium-52 changed the base branch from ojn03/feat-getAllMembers to main October 4, 2023 06:15
Copy link

@kimharr24 kimharr24 left a comment

Choose a reason for hiding this comment

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

🔥 Nice job on your first ticket! Just had a few questions regarding expected fields for MongoDB documents

Copy link

@kimharr24 kimharr24 left a comment

Choose a reason for hiding this comment

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

Nice work! Everything looks good to me - just take out the ObjectId import since we are no longer using Object Ids

@ananyar807 ananyar807 merged commit c56ccc5 into main Oct 23, 2023
chromium-52 added a commit that referenced this pull request Oct 30, 2023
chromium-52 added a commit that referenced this pull request Oct 31, 2023
* worked on getUser method and user parameters
---------

Co-authored-by: Kenny Jung <dukyoung01@naver.com>
@chromium-52 chromium-52 deleted the AR/getUser branch February 22, 2024 22:24
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.

Create getUser endpoint

5 participants