Skip to content

Conversation

@pikurasa
Copy link
Contributor

This PR replaces #511

@sa-fw-an Please review. The http://localhost:5173/assets/tax-filings/2024-Form-990EZ.pdf page is not rendering; what am I missing?

@pikurasa pikurasa requested a review from sa-fw-an October 28, 2025 23:56
@pikurasa pikurasa mentioned this pull request Oct 28, 2025
@github-actions
Copy link

🎉 All Checks Passed!

Status: ✅ Ready to merge

✅ Completed Workflows

Workflow Status Details
🔨 Continuous Integration ✅ Passed Build completed successfully
📝 Code Linting ✅ Passed All formatting and style checks passed

🚀 This PR is ready for review and can be safely merged to main branch!

Great work! Your code meets all quality standards. 👏

@sa-fw-an
Copy link
Member

@pikurasa The directory of the PDF where it is placed is wrong. It must be under public/assets/tax-filings . It works when placed under the mentioned directory.

@pikurasa
Copy link
Contributor Author

@pikurasa The directory of the PDF where it is placed is wrong. It must be under public/assets/tax-filings . It works when placed under the mentioned directory.

I guess I was confused because there are the same 990s under public/assets/990s/.

Given this, should I make a copy in both folders, or should I just move it to tax-filings?

@sa-fw-an
Copy link
Member

@pikurasa The directory of the PDF where it is placed is wrong. It must be under public/assets/tax-filings . It works when placed under the mentioned directory.

I guess I was confused because there are the same 990s under public/assets/990s/.

Given this, should I make a copy in both folders, or should I just move it to tax-filings?

You can move it to tax-filings. It would be better

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