Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Investigate Report Generation #248

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

JennyVong
Copy link
Contributor

@JennyVong JennyVong commented Apr 1, 2024

Notion ticket link

Investigate Report Generation

Implementation description

  • mini demonstration of how a pdf will be generated

Steps to test

  1. go to case overview page of an active case and generate a report
  2. pdf should be dowloaded and viewable

What should reviewers focus on?

Checklist

  • My PR name is descriptive and in imperative tense
  • My commit messages are descriptive and in imperative tense. My commits are atomic and trivial commits are squashed or fixup'd into non-trivial commits
  • I have run the appropriate linter(s)
  • I have requested a review from the PL, as well as other devs who have background knowledge on this PR or who will be building on top of this PR

Copy link

github-actions bot commented Apr 1, 2024

Visit the preview URL for this PR (updated for commit 5ff14d5):

https://childrens-aid-society--pr248-jenny-helios-pdf-gen-iw9rb9yg.web.app

(expires Mon, 08 Apr 2024 01:26:27 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 93f6cb7bff946a1edd29d684408e6d31442cf0b1

Copy link
Contributor

@vaaranan-y vaaranan-y left a comment

Choose a reason for hiding this comment

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

This looks like a viable solution, after having tested it locally! The next steps from here would be to create appropriate styles and elements within a Document component to produce the desired PDF using the design provided in the Figma files. Generate the data dynamically, either using API calls, or useContext if possible. I will update/create a new ticket for the second part of this task.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants