Open rtyley opened 6 months ago
We should pull this into our next health sprint
Steps:
lite
versions of fronts #27389lite
versions of frontsCannot find any metrics for users in the 1% test: https://logs.gutools.co.uk/s/dotcom/goto/7f66b430-65e8-11ef-9afb-57c59e22107a
UPDATE: I had forgot to switch the test on! https://frontend.gutools.co.uk/dev/switchboard
Test switched on at 11:30am 3/9/24
When restarting work on this ticket, give the Fairground team a heads up, as they are working on the fronts and this may affect them
See https://github.com/guardian/frontend/pull/26338#issuecomment-1664215110 - back in November 2017, https://github.com/guardian/frontend/pull/18364 & https://github.com/guardian/frontend/pull/18365 introduced the
/lite.json
representation of fronts:https://github.com/guardian/frontend/blob/8bc978e662702c3972d86d504eeab9b7d2c0c41e/facia/conf/routes#L36-L39
The
lite.json
versions are actually not significantly smaller:...and having 2 versions actually takes up additional memory that we could free up. Consequently I believe the the 'lite' versions should be completely removed.
Things to check...
See if the output of the endpoints that use the 'lite' versions (
renderFrontHeadline()
,renderFrontPressResult()
&renderFrontJsonMinimal()
) stays the same, depending whether they perform their own trimming that will work on the full versions ofPressedPage
, or if were relying on the trimmed nature of the 'lite' version at all.