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

Let users add other users as friends #1056

Merged
merged 58 commits into from
Apr 7, 2024
Merged

Conversation

K02D
Copy link

@K02D K02D commented Dec 2, 2023

Description

Adds screens in the PeerModal to allow users to add other users as Friends

Classmates (existing PeerModal)

image

Find New Friends

image

Current Friends

image

Requests Received

image

Requests Sent

image

Tests

image

TODO

  • Extensively test manually on develop before releasing to production since I wasn't able to test with many different users
  • Need to modify CourseModalBody.tsx so that Semesterly friends, in addition to Facebook friends, are shown as "Friends who have taken this course"
    image

@K02D K02D force-pushed the feature/friends-modal branch from ff232c9 to 3f0e880 Compare December 2, 2023 23:42
@K02D K02D force-pushed the feature/friends-modal branch from 9fa75bc to d6454f2 Compare January 29, 2024 05:05
@K02D K02D changed the title add friends modal Add friends functionality to PeerModal Mar 21, 2024
@K02D K02D changed the title Add friends functionality to PeerModal Add Friends Functionality to PeerModal Mar 21, 2024
@K02D K02D changed the title Add Friends Functionality to PeerModal Add ability for students to add other students as friends Mar 21, 2024
@K02D K02D marked this pull request as ready for review March 21, 2024 02:09
@K02D K02D changed the title Add ability for students to add other students as friends Let users add other users as friends Mar 21, 2024
@K02D K02D requested a review from JiaqiWang18 March 23, 2024 01:42
@JiaqiWang18 JiaqiWang18 merged commit 75e112e into develop Apr 7, 2024
2 of 3 checks passed
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.

2 participants