Skip to content

Add single note display page with details and PDF viewer#135

Open
iffy1234 wants to merge 1 commit intoUTDNebula:developfrom
iffy1234:feature/single-notes-page
Open

Add single note display page with details and PDF viewer#135
iffy1234 wants to merge 1 commit intoUTDNebula:developfrom
iffy1234:feature/single-notes-page

Conversation

@iffy1234
Copy link

@iffy1234 iffy1234 commented Mar 16, 2026

Closes #118

What changed

  • Updated src/app/notes/[id]/page.tsx
  • Replaced the PDF redirect behavior with a rendered single note page
  • Added display for:
    • note name
    • description
    • course
    • section
    • professor
    • author
    • last updated date
  • Embedded the PDF on the page using the browser's native viewer via iframe
  • Added fallbacks for missing description and missing PDF URL

Notes

  • Implemented as a basic render-only page since no design was provided
  • I tested the page locally after configuring the required environment variables

@vercel
Copy link

vercel bot commented Mar 16, 2026

@iffy1234 is attempting to deploy a commit to the Nebula Labs Team on Vercel.

A member of the Team first needs to authorize it.

@iffy1234 iffy1234 closed this Mar 16, 2026
@iffy1234 iffy1234 reopened this Mar 16, 2026
@iffy1234 iffy1234 force-pushed the feature/single-notes-page branch from 0387587 to d3503c9 Compare March 16, 2026 22:35
Copy link
Member

@TyHil TyHil left a comment

Choose a reason for hiding this comment

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

Nice, looking good! In general try not to force push but I think there's just 2 sections of the code that got removed accidentally to be restored and this will be done. You can see the full text of what got removed here: https://github.com/UTDNebula/utd-notebook/pull/135/changes

throw e;
}
}),
byAuthor: protectedProcedure
Copy link
Member

Choose a reason for hiding this comment

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

Restore the byAuthor procedure

Comment on lines -178 to -181
await Promise.all([
callStorageAPI('DELETE', file.id),
ctx.db.delete(files).where(eq(files.id, input.id)),
]);
Copy link
Member

Choose a reason for hiding this comment

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

Restore the Promise.all wrapping these function calls

@vercel
Copy link

vercel bot commented Mar 17, 2026

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

Project Deployment Actions Updated (UTC)
utd-notebook Ready Ready Preview, Comment Mar 17, 2026 8:18pm

Request Review

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.

Single notes page

2 participants