Skip to content

Fix SSR ReferenceError with decorator metadata #855

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

Conversation

JohannesHuster
Copy link

Hi again,

like #853 this PR also solves issue #773. After researching a bit more I found a different solution. Because each solution might have different tradeoffs for you, I will leave both PRs open for now.

I removed the emitDecoratorMetadata option, which is only needed for JIT support, if I am not mistaken: angular/angular#30586 (comment). So with Angular 9 it should not be needed.

The actual issue seems to be a TypeScript limitation, but I opened issue angular/angular#37472 with the angular team, to check whether this is a regression to a workaround they have implemented. From that issue you can find more information from the Angular team in linked issues that describe the original problem very well.

Thank you!

Is only needed for JIT support
@manfredsteyer
Copy link
Owner

Let's go with your other PR.

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