Skip to content

This pull request includes several enhancements and new features related to user profiles and comment input functionality.#16

Merged
ayequill merged 24 commits intomainfrom
nuux
Apr 5, 2024
Merged

This pull request includes several enhancements and new features related to user profiles and comment input functionality.#16
ayequill merged 24 commits intomainfrom
nuux

Conversation

@nuuxcode
Copy link
Copy Markdown
Owner

@nuuxcode nuuxcode commented Apr 5, 2024

Key changes include:

User Profile: A new directory for user profile pages has been created to organize and manage user-related functionalities separately. This includes the implementation of user-related functions in UserController and UserService, such as fetching user information, followers, followings, forums owned by the user, forums where the user is a moderator, posts authored by the user, comments made by the user, votes on posts by the user, votes on comments by the user, and user subscriptions.

Forum Subscription: Implemented forum subscription-related functions in ForumController and ForumService. These functions fetch subscribers for a specific forum, allow users to subscribe to a forum, and allow users to unsubscribe from a forum.

Vote Functionality: Implemented the votePost function in VoteController and VoteService to handle voting on posts.

Comment Input: Fixed a typo in the useFetcher function name in the CommentInput component. Also, added a new CommentInput component to the post page.

Search Functionality: Added search functionality by extracting the search box into a separate component and adding a new page for displaying search results.

Miscellaneous: Made several other minor fixes and enhancements, such as adding optional imageUrl to CreatePostDTO, handling imageUrl in post creation, updating image src to prevent caching, and modifying findAll to fetch all posts by default.

nuuxcode added 24 commits April 4, 2024 02:28
Extracted search box into a separate component.
Added searchPosts function in fetcher hook.
Added a new page for displaying search results.
Uncommented code for full-text and filtered search in post controller.
Made a minor change in post service to not fetch comments when fetching posts.
Deleted the HighlightedPost component as it is no longer needed.
added getUser and getMe functions to useFetcher hook in fetcher.tsx. getUser was
previously named getMe but changed for consistency. this change allows fetching
user data and current user data respectively.
applied styling changes to SearchBox component in search.tsx. added rounded
corners to enhance visual appearance.
…erService

implemented getUser, getUserFollowers, getUserFollowing, getUserOwnedForum,
getUserModeration, getUserPosts, getUserComments, getUserPostVotes,
getUserCommentVotes, and getUserSubscriptions functions in UserController and
UserService. these functions fetch user information, user followers, user
followings, forums owned by the user, forums where the user is a moderator,
posts authored by the user, comments made by the user, votes on posts by the
user, votes on comments by the user, and user subscriptions respectively.
…ontroller and ForumService

implemented getSubscribersForForum, subscribeToForum, and unsubscribeFromForum
functions in ForumController and ForumService. these functions respectively fetch
subscribers for a specific forum, allow users to subscribe to a forum, and allow
users to unsubscribe from a forum.
…vice

implemented votePost function in VoteController and VoteService to handle voting
on posts. this function allows users to vote on posts.
changed 'getUser' function to 'getMe' in the CommentInput component to fix a
typo in the useFetcher function name, ensuring consistency.
created a new directory for user profile pages to organize and manage user-
related functionalities separately.
@nuuxcode nuuxcode requested review from ayequill, omazoz and viictoo April 5, 2024 01:28
@nuuxcode nuuxcode self-assigned this Apr 5, 2024
Copy link
Copy Markdown
Collaborator

@ayequill ayequill left a comment

Choose a reason for hiding this comment

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

🤯🤯🤯

LGTM!!!

@ayequill ayequill merged commit 0756b12 into main Apr 5, 2024
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