Skip to content

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.

@changeset-bot
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

@nx-cloud
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