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
DS-752 Remove ssr-server and all related code (Part 1) #2464
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does the fact that I love seeing deleted code mean I'm getting old? I'll just tell myself it's the fact that the deleted code had incorrect comments and logic bugs. In any case, 😍 .
The code changes make sense to me, and it looks like the build is passing, approved.
@@ -31,7 +31,6 @@ async function getDefaultConfig() { | |||
proxyHostname: configSchema.properties.proxyHostname.default, | |||
proxyHeader: configSchema.properties.proxyHeader.default, | |||
ip, | |||
enableSSR: configSchema.properties.enableSSR.default, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fun fact, there was no default
defined in the schema. I guess it happened to work because it defaulted to something false-y 🤷 .
@@ -125,9 +125,6 @@ program | |||
config.prod = | |||
typeof program.prod === 'undefined' ? config.prod : program.prod; | |||
|
|||
// automatically set enableSSR to true in prod mode and false in dev mode, unless manually set. | |||
config.enableSSR = false; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The comment must be inaccurate? Looks to me like it's disabled in both prod and dev modes...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🔪
Jira
https://pegadigitalit.atlassian.net/browse/DS-752
Summary
Remove ssr-server and all other experimental SSR code.
Details
SSR was an experiment to pass HTML to a back-end service and get back rendered web component HTML. It mostly worked but was never used and isn't needed going forward. We're removing it as part of repo clean up.
How to test