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 9 commits into
base: master
Choose a base branch
from
Draft

Feat/howto detail ssr #3898

wants to merge 9 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 #6357

Run Properties:  status check failed Failed #6357  •  git commit ab0ffce7aa: fix: build
Project onearmy-community-platform
Run status status check failed Failed #6357
Run duration 13m 48s
Commit git commit ab0ffce7aa: fix: build
Committer Mário Nunes
View all properties for this run ↗︎

Test results
Tests that failed  Failures 26
Tests that were flaky  Flaky 0
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 49

Tests for review

Failed  howto/write.spec.ts • 3 failed tests • ci-chrome

View Output Video

Test Artifacts
[How To] > [Create a how-to] > [By Authenticated] Test Replay Screenshots Video
[How To] > [Edit a how-to] > [By Authenticated] Test Replay Screenshots Video
[How To] > [Edit a how-to] > [By Owner] Test Replay Screenshots Video
Failed  research/write.spec.ts • 3 failed tests • ci-chrome

View Output Video

Test Artifacts
[Research] > [Create research article] > [By Authenticated] Test Replay Screenshots Video
[Research] > [Create research article] > [Any PP user] Test Replay Screenshots Video
[Research] > [Displays draft updates for Author] > [By Authenticated] Test Replay Screenshots Video
Failed  notifications.spec.ts • 2 failed tests • ci-chrome

View Output Video

Test Artifacts
[Notifications] > [are not generated when the howTo author is triggering notification] Test Replay Screenshots Video
[Notifications] > [are generated by clicking on useful for how-tos] Test Replay Screenshots Video
Failed  SignUp.spec.ts • 2 failed tests • ci-chrome

View Output Video

Test Artifacts
[User sign-up] > [Cannot duplicate existing user] > Prevents duplicate name Test Replay Screenshots Video
[User sign-up] > [Cannot duplicate existing user] > Prevents duplicate email Test Replay Screenshots Video
Failed  questions/discussions.spec.ts • 1 failed test • ci-chrome

View Output Video

Test Artifacts
[Questions.Discussions] > allows authenticated users to contribute to discussions Test Replay Screenshots Video

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

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.

1 participant