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

Updated deprecated API endpoint from /users/self to /users?profile=true #27

Conversation

Hariom01010
Copy link
Contributor

Date: 09 Dec 2024

Developer Name: Hariom Vashista


Issue Ticket Number

Fixes: Real-Dev-Squad/todo-action-items#255 repo#12

Description

This PR aims to change the documentation and update the /users/self API endpoint to /users?profile=true

Documentation Updated?

  • Yes
  • No

Under Feature Flag

  • Yes
  • No

Database Changes

  • Yes
  • No

Breaking Changes

  • Yes
  • No

Development Tested?

  • Yes
  • No

Screenshots

Screenshot 1

Test Coverage

Screenshot 1

Additional Notes

Copy link
Contributor

@prakashchoudhary07 prakashchoudhary07 left a comment

Choose a reason for hiding this comment

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

Is there a scope for improving the document?

@Hariom01010
Copy link
Contributor Author

I feel sir, the documentation in it's current state is easy to understand and follow for a beginner like me. So i won't suggest any changes as of now (one of the reasons is also that i am not familiar with the backend projects). So i have updated the API endpoint mentioned in the links. But surely in future, if i find something which would improve this documentation would suggest those changes.

@prakashchoudhary07
Copy link
Contributor

I feel sir, the documentation in it's current state is easy to understand and follow for a beginner like me. So i won't suggest any changes as of now (one of the reasons is also that i am not familiar with the backend projects). So i have updated the API endpoint mentioned in the links. But surely in future, if i find something which would improve this documentation would suggest those changes.

See if you can get in touch with 2-3 people who did then setup and get their feedback on it

@Hariom01010
Copy link
Contributor Author

Hariom01010 commented Dec 12, 2024

Sure sir will look into it and update here

@iamitprakash iamitprakash merged commit 0681b26 into Real-Dev-Squad:main Dec 29, 2024
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.

Remove /users/self and implement a new API Endpoint
5 participants