Skip to content

Conversation

@nsemets
Copy link
Collaborator

@nsemets nsemets commented Jan 15, 2026

  • Ticket: [ENG-10032]
  • Feature flag: n/a

Summary of Changes

  1. Added SSR setup.
  2. Added new routes for SSR.
  3. Fixed issues with SSR.

@nsemets nsemets requested a review from brianjgeiger January 15, 2026 11:56
@brianjgeiger brianjgeiger changed the title [ENG-10032] Main resources page [ENG-10032] SSR: Main resources page Jan 23, 2026
Copy link
Contributor

@brianjgeiger brianjgeiger left a comment

Choose a reason for hiding this comment

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

Don't merge this; we'll test it on a staging server by itself first, then merge it to develop when the time comes.

I'm not sure if the addition of the safeHtml pipe and the removal of the decodeHtml pipe was the correct thing to do, so I'll be having QA test that explicitly to make sure. We may have needed those to remain the way they were, unless you know of explicit bugs that we have related to those components.

@nsemets
Copy link
Collaborator Author

nsemets commented Jan 23, 2026

I'm not sure if the addition of the safeHtml pipe and the removal of the decodeHtml pipe was the correct thing to do.

There was an issue with SSR related to decodeHtml in one component, so I removed it entirely. It was duplicating the functionality of safeHtml, which is why it was removed.

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