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

re-order components and address components loading at different point #208

Merged
merged 3 commits into from
Oct 2, 2024

Conversation

egauzens
Copy link
Collaborator

@egauzens egauzens commented Oct 1, 2024

Addressing this ticket: https://www.wrike.com/open.htm?id=1504551274

And seeing about loading the entire homepage via SSR instead of having some rendered on the client. If i recall we had issues with this on deployment before eventhough it works fine locally but I am trying it out on staging again

Copy link

cypress bot commented Oct 1, 2024

SPARC Vue3 Portal Testing    Run #279

Run Properties:  status check failed Failed #279  •  git commit cfa9445866: update tests
Project SPARC Vue3 Portal Testing
Branch Review re-order-homepage-components
Run status status check failed Failed #279
Run duration 15m 01s
Commit git commit cfa9445866: update tests
Committer Eric Gauzens
View all properties for this run ↗︎

Test results
Tests that failed  Failures 1
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 1
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 71
View all changes introduced in this branch ↗︎

Tests for review

Failed  tests/cypress/e2e/homepage.cy.js • 1 failed test

View Output

Test Artifacts
Homepage > SPARC by the numbers Test Replay Screenshots

@egauzens
Copy link
Collaborator Author

egauzens commented Oct 2, 2024

The latest test failure is not to do with the changes, but rather the fact that the metrics have not been published. I have notified dat-core

@egauzens egauzens merged commit eb18d69 into main Oct 2, 2024
0 of 2 checks passed
@egauzens egauzens deleted the re-order-homepage-components branch October 2, 2024 12:51
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.

1 participant