Skip to content

Export Single Graph to PDF 📂 - Will & Justin #48#53

Open
woleary2 wants to merge 3 commits intomainfrom
48-sprint-4b-export-single-graph-to-pdf---will-justin
Open

Export Single Graph to PDF 📂 - Will & Justin #48#53
woleary2 wants to merge 3 commits intomainfrom
48-sprint-4b-export-single-graph-to-pdf---will-justin

Conversation

@woleary2
Copy link
Collaborator

@woleary2 woleary2 commented Feb 1, 2026

Sprint

Export Single Graph to PDF 📂 - Will & Justin #48

Who worked on this sprint?

Will and Justin

Features Implemented

Exporting a bar/line graph to a pdf

New files created

/src/lib/exporttopdf.ts

Existing files modified

/src/app/graphs/page.tsx
/src/components/BarGraph.tsx
/src/components/LineGraph.tsx

Acceptance Criteria

The export button downloads the graph in current view to a PDF

PDF does not cut-off any part of the graph (i.e. looks identical to browser)

Filters are captured in the exported PDF

Exporting works across all major browsers

Export button is responsive while loading and resets once finished

UI matches Figma

Code runs locally without errors and builds successfully

Merge via PR

Testing: how did you test?

Tried exporting a variety of bar/line graphs on different browsers

Features Not Implemented/Incomplete

Loading feature is not developed

Screenshots:

https://github.com/user-attachments/assets/3e07764d-a093-41ea-8fda-49cf29ce59cc

Tag Dan and Shayne

@danglorioso @shaynesidman

@woleary2 woleary2 added enhancement New feature or request Sprint labels Feb 1, 2026
@woleary2 woleary2 linked an issue Feb 1, 2026 that may be closed by this pull request
8 tasks
@vercel
Copy link

vercel bot commented Feb 1, 2026

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

Project Deployment Actions Updated (UTC)
mhd Error Error Feb 1, 2026 7:17pm

Copy link
Collaborator

@tika tika left a comment

Choose a reason for hiding this comment

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

just a few things that you need to uncomment

"dotenv": "^17.2.3",
"drizzle-orm": "^0.44.7",
"framer-motion": "^12.23.24",
"html2canvas": "^1.4.1",
Copy link
Collaborator

Choose a reason for hiding this comment

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

why install a dep if you're not going to use it

Copy link
Collaborator

Choose a reason for hiding this comment

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

nit: call this file export-to-pdf.ts. it's cleaner

Copy link
Collaborator

Choose a reason for hiding this comment

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

why


import { useEffect, useState } from "react";
import { toast } from "sonner";
//import { toast } from "sonner";
Copy link
Collaborator

Choose a reason for hiding this comment

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

why

//const isAuthPage = pathname === "/signin";

// If on auth pages, just render children without sidebar
if (isAuthPage) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

why

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request Sprint

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Sprint 4b]: Export Single Graph to PDF 📂 - Will & Justin

3 participants