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

fetch users' posts #79

Closed
wants to merge 2 commits into from
Closed

fetch users' posts #79

wants to merge 2 commits into from

Conversation

jayKhakhariya
Copy link
Collaborator

@jayKhakhariya jayKhakhariya commented Mar 20, 2021

  • This pull request displays posts of the user that's logged in;
  • These posts will be displayed on the profile page which is in progress

EDIT: Tests will soon be written for the backend; just tested it with a frontend basic button.

Copy link
Collaborator

@humayraR humayraR left a comment

Choose a reason for hiding this comment

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

You maybe will be fetching the posts of the wrong user

gamersnet_backend/routes/posts/getPosts.js Outdated Show resolved Hide resolved
gamersnet_backend/routes/posts/getPosts.js Outdated Show resolved Hide resolved
@humayraR humayraR self-requested a review March 20, 2021 18:20
@jayKhakhariya jayKhakhariya self-assigned this Mar 21, 2021
@jayKhakhariya
Copy link
Collaborator Author

This pull request is getting closed without merging the commits because other PR already has these changes

@humayraR humayraR linked an issue Mar 25, 2021 that may be closed by this pull request
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.

Post Management
2 participants