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

23 [Part 1]: Unfollowing a local author #113

Merged
merged 6 commits into from
Nov 23, 2023
Merged

23 [Part 1]: Unfollowing a local author #113

merged 6 commits into from
Nov 23, 2023

Conversation

kaynzhel
Copy link
Contributor

Changes

  • On hover, when user is already following an author, we see an "Unfollow" button
  • Adding unfollow modal and calling the DELETE method
    • Modal is the same as the DeletePostModal in terms of its design
  • Updated the DELETE method on the backend to also delete Follow and InboxItem records since user has already unfollowed. (They can refollow again)

Example (Following to unfollowing)

Screen.Recording.2023-11-20.at.1.46.01.PM.mov

@kaynzhel kaynzhel mentioned this pull request Nov 23, 2023
@kaynzhel kaynzhel linked an issue Nov 23, 2023 that may be closed by this pull request
@kaynzhel
Copy link
Contributor Author

rebased latest main

@rmgutierrez
Copy link
Contributor

Should we either centre "Unfollow authorName" or align both texts with each other?

image

Copy link
Contributor

@rmgutierrez rmgutierrez left a comment

Choose a reason for hiding this comment

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

Apart from my comment, everything seems to be working properly!

@rmgutierrez rmgutierrez merged commit 6d97492 into main Nov 23, 2023
1 check passed
@rmgutierrez rmgutierrez deleted the 23-unfollow branch November 23, 2023 06:01
rmgutierrez added a commit that referenced this pull request Nov 23, 2023
## Review #113 First

### Changes
- Added endpoint to easily delete a `Follow` object and its
corresponding `InboxItem` object
- Implement declining a friend request
  - Declining means that it gets remove from the inbox 
- Adding these buttons made the rendering more noticeable, so I
implement a `Loading` component to be shown when data is still being
fetched.

### Example


https://github.com/uofa-cmput404/404f23project-404-team-not-found/assets/40973251/d7313336-a4a9-4fce-99b3-cfc367055767


https://github.com/uofa-cmput404/404f23project-404-team-not-found/assets/40973251/4693cdfb-4a2f-449f-bafd-a2e77785268f
@kaynzhel kaynzhel linked an issue Nov 23, 2023 that may be closed by this pull request
@kaynzhel kaynzhel mentioned this pull request Nov 25, 2023
kaynzhel added a commit that referenced this pull request Nov 25, 2023
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.

As an author, I want un-befriend local and remote authors
2 participants