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

Profile page / Follower + Following #49

Merged
merged 10 commits into from
Apr 18, 2024
Merged

Profile page / Follower + Following #49

merged 10 commits into from
Apr 18, 2024

Conversation

jakobphilippe
Copy link
Contributor

Description

Change below here

Updates profile page so that followers and following work. Still needs to be integrated with Clerk 100%. Currently, the logged in user is the clerk user but the following/follower pages are using the DB user. We can simply look up the user from clerk by ID but didn't have time to do this yet. Functionality wise though everything is working though, although the logged in user first/last name is still undefined (at least for me because my clerk user doesn't have a first and last name).

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. If they are unit tests, provide the file name the tests are in. If they are not unit tests, describe how you tested the change.

Change below here. If this is a frontend MR, include screenshots

Add here

Checklist

  • I have performed a self-review of my code
  • I have reached out to another developer to review my code
  • I have commented my code, particularly in hard-to-understand areas
  • New and existing unit tests pass locally with my changes

@capture120 capture120 self-requested a review April 17, 2024 23:21
@capture120
Copy link
Contributor

looking very clean

@leoRysing leoRysing merged commit 586bd1d into main Apr 18, 2024
5 checks passed
@leoRysing leoRysing deleted the profile-followers branch April 18, 2024 00:40
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.

5 participants