Skip to content

Added more e2e tests for the workflows and added ai summary feature f…#14

Merged
Ragyudy merged 2 commits into
mainfrom
ai-summary-feature
Dec 4, 2025
Merged

Added more e2e tests for the workflows and added ai summary feature f…#14
Ragyudy merged 2 commits into
mainfrom
ai-summary-feature

Conversation

@emmettxs

@emmettxs emmettxs commented Dec 4, 2025

Copy link
Copy Markdown
Contributor

…or each document

Summary

I added more e2e tests for the basic workflows including navigating to pages and making sure buttons and texts existed for certain pages. I also added a new feature to the document view page. It summarizes the documents with OpenAI so the user can easily read a 2-3 sentence summary of the document.

Related Issues (Optional)

  • Closes #123
  • Fixes #456
  • Relates to #789

Changes

  • Added more e2e tests for our website in general
  • Added a new feature. It uses OpenAI API key to summarize the documents the user uploaded. Each document viewer now has an AI summary of the document.
  • Refactor (non-breaking code change that improves readability/structure)
  • Performance Improvement
  • Security Improvement
  • Documentation Update
  • Test Update / New Tests
  • Configuration / CI / Build Change
  • Other (please describe)

Main Contributors

Emmett Seto

Screenshots (if applicable)

Checklist

  • PR targets the main branch
  • At least one reviewer assigned
  • Tests added or updated
  • Documentation updated (if needed)
  • All conversations resolved
  • Linked issues are referenced above (if applicable)

Notes for Reviewers

Can you check my logic for my ai summarizer function

@emmettxs emmettxs requested a review from Ragyudy December 4, 2025 01:43
@vercel

vercel Bot commented Dec 4, 2025

Copy link
Copy Markdown

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

Project Deployment Preview Comments Updated (UTC)
f25-inki Error Error Dec 4, 2025 1:52am

@github-actions

github-actions Bot commented Dec 4, 2025

Copy link
Copy Markdown

Coverage

Unit + Integration coverage: 84.27%

@Ragyudy Ragyudy merged commit 7a089f8 into main Dec 4, 2025
7 of 8 checks passed
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.

2 participants