Skip to content

Conversation

@longshuicy
Copy link
Collaborator

@longshuicy longshuicy commented Oct 14, 2025

11052025 update:

Switch this back to draft since we need to discuss what's the behavior for a empty folder not associate with any conversation.

When creating a folder, the request body includes the courseName, but the folders table doesn’t have a course_name field, so it isn’t stored. Since conversations link to folders via folder_id, how is the folder tied to a specific course before any conversations exist?


To test:

  • Create a folder under one project, drag some conversation to that folder
  • Open another project, the folders shouldn't be visible

@longshuicy longshuicy requested a review from rohan-uiuc October 14, 2025 18:50
@vercel
Copy link

vercel bot commented Oct 14, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
uiuc-chat-frontend Ready Ready Preview Comment Oct 14, 2025 6:51pm

async function handler(req: AuthenticatedRequest, res: NextApiResponse) {
const { method } = req
const user_email = req.user?.email as string | undefined
const courseName = req.query.courseName as string
Copy link
Collaborator

Choose a reason for hiding this comment

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

In post call while creating folder, the request is giving 200 but sends course_name as undefined because it will be in the body which causes an instant disappearing of the folder.

Server debug logs for the post call:

eq.params {}
courseName undefined
user_email rohan13@illinois.edu
method POST
req.body {
  folder: {
    id: '6b06dde6-6100-4913-985c-4793bdfca7d1',
    name: 'New folder',
    type: 'chat'
  },
  courseName: 'test'
}

CleanShot 2025-11-05 at 14 43 17

@longshuicy longshuicy marked this pull request as draft November 5, 2025 22:38
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.

3 participants