Skip to content
This repository has been archived by the owner on Dec 8, 2021. It is now read-only.

Feature/friends #166

Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

Feature/friends #166

wants to merge 14 commits into from

Conversation

devnoll
Copy link
Contributor

@devnoll devnoll commented Feb 8, 2021 •
edited
Loading

  • Friend List of all friends(accepted, pending, requested) on the User's Account
  • Method of adding friends
  • Method of accepting or declining friend requests
  • Method of removing friends

closes #6

@devnoll devnoll requested a review from a team February 8, 2021 00:49
@devnoll devnoll linked an issue Feb 8, 2021 that may be closed by this pull request
Friends #6
Open
@devnoll devnoll marked this pull request as draft February 10, 2021 10:55
@devnoll devnoll marked this pull request as ready for review February 10, 2021 13:22
@devnoll devnoll requested review from a team and ConnorJC3 and removed request for a team February 12, 2021 04:19
@devnoll devnoll marked this pull request as draft February 12, 2021 10:02
@devnoll devnoll marked this pull request as ready for review February 13, 2021 09:28
@ConnorJC3
Copy link
Member

@TorresEd we really need a test for this in CI, but schema.rb is not up to date with your latest changes and it's incorrectly causing CI to pass when it should crash and burn

@ConnorJC3 ConnorJC3 mentioned this pull request Feb 13, 2021
Closed
@devnoll devnoll marked this pull request as draft February 14, 2021 18:54
@devnoll devnoll marked this pull request as ready for review February 14, 2021 21:10
@devnoll devnoll force-pushed the feature/friends branch 2 times, most recently from 2443920 to edf2b91 Compare February 18, 2021 04:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@ConnorJC3 ConnorJC3 Awaiting requested review from ConnorJC3

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Friends
2 participants
@devnoll @ConnorJC3