Skip to content

Add Cypress failure screenshot docs#38031

Merged
juan-fernandez merged 7 commits into
masterfrom
juan-fernandez/cypress-failure-screenshots
Jul 8, 2026
Merged

Add Cypress failure screenshot docs#38031
juan-fernandez merged 7 commits into
masterfrom
juan-fernandez/cypress-failure-screenshots

Conversation

@juan-fernandez

@juan-fernandez juan-fernandez commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

What does this PR do? What is the motivation?

Adds guidance for uploading Cypress test failure screenshots to Datadog in the JavaScript and TypeScript test setup documentation.

https://docs-staging.datadoghq.com/juan-fernandez/cypress-failure-screenshots/tests/setup/javascript/?tab=cypress#test-failure-screenshot-upload

Merge readiness

  • Ready for merge

For Datadog employees:

  • ⚠️ Your branch name MUST follow the <name>/<description> convention and include the forward slash (/). If you've already created your PR with an incorrect branch name, please rename your branch and open a fresh PR.
  • 🤖 New: Comment with /review to run an automated check that catches common issues before a Documentation team member reviews your PR.

AI assistance

Used Codex to draft and apply the documentation update.

Additional notes

Validation:

  • git diff --check --cached -- content/en/tests/setup/javascript.md
  • vale content/en/tests/setup/javascript.md (0 errors; existing warnings and suggestions remain in unchanged page content)

@juan-fernandez juan-fernandez marked this pull request as ready for review July 8, 2026 11:14
@juan-fernandez juan-fernandez requested review from a team as code owners July 8, 2026 11:14
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Preview links (active after the build_preview check completes)

Modified Files

…ess-failure-screenshots

# Conflicts:
#	content/en/tests/setup/javascript.md
@juan-fernandez

Copy link
Copy Markdown
Contributor Author

/review

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🤖 Automated review by Claude. AI-generated; verify before acting.

Small addition documenting a new Cypress screenshot upload feature. Two minor suggestions: heading could be more actionable, and users would benefit from knowing where uploaded screenshots appear in Datadog to verify the setup worked.

Reviewed bda2bffafe227200a9c20ec7de1ec16e6a73c35cworkflow run

Comment thread content/en/tests/setup/javascript.md Outdated
Comment thread content/en/tests/setup/javascript.md Outdated
@estherk15 estherk15 changed the title [DOCS-XXXXX] Add Cypress failure screenshot docs Add Cypress failure screenshot docs Jul 8, 2026

@estherk15 estherk15 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for the update!

juan-fernandez and others added 2 commits July 8, 2026 15:48
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@github-actions github-actions Bot added the Images Images are added/removed with this PR label Jul 8, 2026
@juan-fernandez juan-fernandez added the WORK IN PROGRESS No review needed, it's a wip ;) label Jul 8, 2026
@juan-fernandez

Copy link
Copy Markdown
Contributor Author

/review

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🤖 Automated review by Claude. AI-generated; verify before acting.

Small, focused addition. Content is clear and the reference link + image are set up correctly. One minor wording nit on the enablement instructions.

Reviewed 50e0f22dfdc79b36223b9b0791afcb82a469a47fworkflow run

Comment thread content/en/tests/setup/javascript.md Outdated
juan-fernandez and others added 3 commits July 8, 2026 16:00
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
…creenshots' into juan-fernandez/cypress-failure-screenshots
@juan-fernandez juan-fernandez removed the WORK IN PROGRESS No review needed, it's a wip ;) label Jul 8, 2026
@juan-fernandez juan-fernandez merged commit 04fce82 into master Jul 8, 2026
19 checks passed
@juan-fernandez juan-fernandez deleted the juan-fernandez/cypress-failure-screenshots branch July 8, 2026 19:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Images Images are added/removed with this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants