Conversation
- Implement folder page to display user folders - Add folder card component for folder representation - Added 'add to folder' button in apartment page - Debugged authentication issues in folder operations
|
Lauren Pothuru seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account. You have signed the CLA already but the status is still pending? Let us recheck it. |
|
[diff-counting] Significant lines: 1911. This diff might be too big! Developer leads are invited to review the code. |
|
Super cool feature Lauren! Your backend routes look really polished and nicely organized. You followed our coding conventions really well, which made understanding your work 10x easier. The UI from the screenshots above also looks really nice. Nice job! |
There was a problem hiding this comment.
Great work. Code structure is organized, well-documented, and the inline component comments are helpful (e.g. AddToFolderPopover.tsx)
Some future points of consideration for feature expansions:
- Shared folder ownership: The current document structure for folders uses a single user value, which will need to change to support co-ownership.
- Proposed solution: Replace the string-based user field with an array to accommodate multiple users.
Implementation requirements:
- change to user-id based query logic (@route GET /api/folders)
- persistence logic for multiple owners (e.g. modified deletion in @route DELETE /api/folders/:folderId to remove specific users rather than deleting the entire folder).
These will be future requirements. Current implementation is great. Well done.
Summary
This pull request implements the folder saving feature, where users can assign apartments to custom folders.
Test Plan
Screenshots of the Folders page and Save to Folder modal



Postman was used to test the api endpoints and make sure the backend worked smoothly.