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

Feat/howto detail ssr #3898

Draft
wants to merge 30 commits into
base: master
Choose a base branch
from
Draft

Feat/howto detail ssr #3898

wants to merge 30 commits into from

Conversation

mariojsnunes
Copy link
Contributor

@mariojsnunes mariojsnunes commented Oct 7, 2024

PR Checklist

PR Type

What kind of change does this PR introduce?

  • Feature (adds functionality)
  • Refactoring (no functional changes)

What is the current behavior?

Howto page is rendered client-side.

What is the new behavior?

Howto page is rendered server-side.
More notable changes:

  • EnvironmentContext

Does this PR introduce a breaking change?

  • Yes
  • No

Git Issues

Closes #

Copy link

cypress bot commented Oct 11, 2024

onearmy-community-platform    Run #6378

Run Properties:  status check failed Failed #6378  •  git commit a9a2d8ce6f: fix: profile views
Project onearmy-community-platform
Run status status check failed Failed #6378
Run duration 08m 57s
Commit git commit a9a2d8ce6f: fix: profile views
Committer Mário Nunes
View all properties for this run ↗︎

Test results
Tests that failed  Failures 16
Tests that were flaky  Flaky 3
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 57

Tests for review

Failed  howto/read.spec.ts • 8 failed tests • ci-chrome

View Output Video

Test Artifacts
... > [See all info] Test Replay Screenshots Video
... > [Allows opening of attachments] Test Replay Screenshots Video
... > [Delete button is visible] Test Replay Screenshots Video
... > [Edit button is visible] Test Replay Screenshots Video
... > [Delete button is visible] Test Replay Screenshots Video
... > [Marked for deletion message] Test Replay Screenshots Video
... > [Delete Button is disabled] Test Replay Screenshots Video
... > [Delete Button is disabled] Test Replay Screenshots Video
Failed  howto/discussions.spec.ts • 2 failed tests • ci-chrome

View Output Video

Test Artifacts
[Howto.Discussions] > can open using deep links Test Replay Screenshots Video
[Howto.Discussions] > allows authenticated users to contribute to discussions Test Replay Screenshots Video
Failed  notifications.spec.ts • 1 failed test • ci-chrome

View Output Video

Test Artifacts
[Notifications] > [are generated by clicking on useful for how-tos] Test Replay Screenshots Video
Failed  howto/write.spec.ts • 1 failed test • ci-chrome

View Output Video

Test Artifacts
[How To] > [Create a how-to] > [By Authenticated] Test Replay Screenshots Video
Failed  profile.spec.ts • 1 failed test • ci-chrome

View Output Video

Test Artifacts
[By Beta Tester] > [Displays view count for profile with views] Test Replay Screenshots Video

The first 5 failed specs are shown, see all 8 specs in Cypress Cloud.

Flakiness  write.spec.ts • 2 flaky tests • ci-chrome

View Output Video

Test Artifacts
[Research] > [Create research article] > [Warning on leaving page] Test Replay Screenshots Video
[Research] > [Displays draft updates for Author] > [By Authenticated] Test Replay Screenshots Video
Flakiness  follow.spec.ts • 1 flaky test • ci-chrome

View Output Video

Test Artifacts
[Research] > [By Authenticated] > [Follow button] Test Replay Screenshots Video

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Status: 🆕 New
Development

Successfully merging this pull request may close these issues.

2 participants