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

fix: currentUser issue169 #196

Merged
merged 3 commits into from
Jan 20, 2024
Merged

fix: currentUser issue169 #196

merged 3 commits into from
Jan 20, 2024

Conversation

IrfanshaikH-7
Copy link
Contributor

@IrfanshaikH-7 IrfanshaikH-7 commented Jan 18, 2024

Related Issue

Issue #169

Description:

Fix the issue where in user profile, it was previously rendering currently logged in user to profile owner's details.

Changes:

  • Change the user query handler from getcurrentUser to getSingleUser.
  • Added types for user
  • Changed the query to indexes
  • refactored and removed unused import

Copy link

vercel bot commented Jan 18, 2024

Someone is attempting to deploy a commit to a Personal Account owned by @Sanchitbajaj02 on Vercel.

@Sanchitbajaj02 first needs to authorize it.

@github-actions github-actions bot added wait for reviewers SWOC S4 Social Winter of Code (S4) 2024 contribution tag labels Jan 18, 2024
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Great job, @IrfanshaikH-7! 🎉 Thank you for submitting your pull request. Your contribution is valuable and we appreciate your efforts to improve our project. Keep being the part of the community!

'We will promptly review your changes and offer feedback. Keep up the excellent work!
Kindly remember to check our Contributing Guidelines'

@Sanchitbajaj02 Sanchitbajaj02 added 🪲 bug Something isn't working 🛠 goal: fix The label is related to fixing a bug MEDIUM Difficulty Level: Medium and removed wait for reviewers labels Jan 18, 2024
src/types/index.d.ts Outdated Show resolved Hide resolved
src/types/index.d.ts Show resolved Hide resolved
appwrite-gen/schema.json Outdated Show resolved Hide resolved
src/components/pages/user/userPosts/index.tsx Outdated Show resolved Hide resolved
@Sanchitbajaj02 Sanchitbajaj02 merged commit f141b0e into Sanchitbajaj02:master Jan 20, 2024
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🪲 bug Something isn't working 🛠 goal: fix The label is related to fixing a bug MEDIUM Difficulty Level: Medium SWOC S4 Social Winter of Code (S4) 2024 contribution tag
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: User Page rendering "currentUser"'s details rather than the posts owners
2 participants