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(platform-server): align server renderer interface with base renderer #47868
Conversation
The `ServerRenderer` wasn't aligned with the `Renderer2` interface which meant that it was still referring to the old `debugInfo` parameters. It also wasn't implementing the `preserveContent` argument of `selectRootElement` which can lead to incosistencies between the server and the client. Fixes angular#47844.
@alan-agius4 @crisbeto is there any test we could put in place to prevent this type of de-synchronisation happening in the future? |
I think that normally this would've been caught by TS through the |
This PR was merged into the repository by commit 54e32da. |
…rer (#47868) The `ServerRenderer` wasn't aligned with the `Renderer2` interface which meant that it was still referring to the old `debugInfo` parameters. It also wasn't implementing the `preserveContent` argument of `selectRootElement` which can lead to incosistencies between the server and the client. Fixes #47844. PR Close #47868
…rer (angular#47868) The `ServerRenderer` wasn't aligned with the `Renderer2` interface which meant that it was still referring to the old `debugInfo` parameters. It also wasn't implementing the `preserveContent` argument of `selectRootElement` which can lead to incosistencies between the server and the client. Fixes angular#47844. PR Close angular#47868
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
…rer (angular#47868) The `ServerRenderer` wasn't aligned with the `Renderer2` interface which meant that it was still referring to the old `debugInfo` parameters. It also wasn't implementing the `preserveContent` argument of `selectRootElement` which can lead to incosistencies between the server and the client. Fixes angular#47844. PR Close angular#47868
The
ServerRenderer
wasn't aligned with theRenderer2
interface which meant that it was still referring to the olddebugInfo
parameters. It also wasn't implementing thepreserveContent
argument ofselectRootElement
which can lead to incosistencies between the server and the client.Fixes #47844.