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

fix[angular-gen2]: add angular 19 ssr e2e and fix hydration errors #3806

Merged
merged 10 commits into from
Jan 24, 2025

Conversation

sidmohanty11
Copy link
Contributor

@sidmohanty11 sidmohanty11 commented Dec 30, 2024

Description

Fixes the hydration error thrown from angular for using ngComponentOutlet, adds angular-19-ssr to e2e and an example of how users can override fetch to use HTTP client from angular that should help in caching HTTP requests

Jira
https://builder-io.atlassian.net/browse/ENG-7946
https://builder-io.atlassian.net/browse/ENG-7945
https://builder-io.atlassian.net/browse/ENG-7665

Screenshot
If relevant, add a screenshot or two of the changes you made.

Copy link

changeset-bot bot commented Dec 30, 2024

🦋 Changeset detected

Latest commit: 666cc89

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@builder.io/sdk-angular Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link

nx-cloud bot commented Dec 30, 2024

View your CI Pipeline Execution ↗ for commit 666cc89.

Command Status Duration Result
nx test @e2e/nextjs-sdk-next-app ✅ Succeeded 8m 37s View ↗
nx test @e2e/qwik-city ✅ Succeeded 8m 14s View ↗
nx test @e2e/sveltekit ✅ Succeeded 6m 59s View ↗
nx test @e2e/nuxt ✅ Succeeded 8m 9s View ↗
nx test @e2e/solid ✅ Succeeded 6m 18s View ↗
nx test @e2e/remix ✅ Succeeded 6m 7s View ↗
nx test @e2e/vue ✅ Succeeded 5m 54s View ↗
nx test @e2e/svelte ✅ Succeeded 5m 54s View ↗
Additional runs (35) ✅ Succeeded ... View ↗

☁️ Nx Cloud last updated this comment at 2025-01-23 09:02:59 UTC

@sidmohanty11
Copy link
Contributor Author

@samijaber can you please take a look

Copy link
Contributor

@samijaber samijaber left a comment

Choose a reason for hiding this comment

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

🙏🏽

/**
* Angular doesn't support hydration for components created dynamically.
* Refer: https://angular.dev/errors/NG0503
* GitHub issue: https://github.com/angular/angular/issues/51798
Copy link
Contributor

Choose a reason for hiding this comment

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

Does this issue resolve content duplication as well like https://builder-io.atlassian.net/browse/ENG-8042 ?

@sidmohanty11 sidmohanty11 merged commit e07fcf0 into BuilderIO:main Jan 24, 2025
51 checks passed
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.

2 participants