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

Add Export to PDF for Score Reports #394

Merged
merged 4 commits into from
Feb 29, 2024
Merged

Add Export to PDF for Score Reports #394

merged 4 commits into from
Feb 29, 2024

Conversation

lucasxsong
Copy link
Contributor

@lucasxsong lucasxsong commented Feb 28, 2024

Exports overview graphs, task descriptions, and task-specific graphs into a pdf.
image

Copy link

github-actions bot commented Feb 28, 2024

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

https://roar-staging--pr394-enh-report-print-vie-nc47o7ar.web.app

(expires Thu, 07 Mar 2024 12:16:45 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 2631e9c58fd0104ecbfddd72a62245ddac467460

Copy link

cypress bot commented Feb 28, 2024

1 flaky test on run #346 ↗︎

0 24 0 0 Flakiness 1

Details:

Tests for PR 394 "Add Export to PDF for Score Reports" from commit "480461318d95...
Project: roar-dashboard-e2e Commit: 480461318d
Status: Passed Duration: 21:17 💡
Started: Feb 29, 2024 12:18 PM Ended: Feb 29, 2024 12:39 PM
Flakiness  cypress/e2e/participant/default-tests/playSWR.cy.js • 1 flaky test

View Output

Test Artifacts
Testing playthrough of SWR as a participant > ROAR-Word Playthrough Test Test Replay Screenshots

Review all test suite changes for PR #394 ↗︎

@lucasxsong lucasxsong requested review from kellyel and richford and removed request for kellyel February 29, 2024 00:19
@lucasxsong lucasxsong merged commit 34f7fd9 into main Feb 29, 2024
13 of 14 checks passed
@lucasxsong lucasxsong deleted the enh/report-print-view branch February 29, 2024 19:43
Copy link

sentry-io bot commented Mar 4, 2024

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

Did you find this useful? React with a 👍 or 👎

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.

None yet

2 participants