guardian / frontend

The Guardian DotCom.
https://theguardian.com
Other
5.82k stars 556 forks source link

Rename ab test with extra dash to match convention #27274

Closed sookburt closed 5 days ago

sookburt commented 6 days ago

What is the value of this and can you measure success?

A tiny change to fix the ab test which is not working from the PR27261

What does this change?

It adds a dash to the test name in ABTestSwitches.scala to match the slightly different ab test name elsewhere in the frontend code where the blog part is title case accidentally (like StickyLiveBlogAskTest).

Screenshots

No visible change other than the sticky liveblog showing for the variant - see DCR PR11723 which contains screenshots.

Checklist

github-actions[bot] commented 6 days ago

Deploy build 2308 of dotcom:frontend-all to CODE

All deployment options - [Deploy build 2308 of `dotcom:frontend-all` to CODE](https://riffraff.gutools.co.uk/deployment/deployAgain?project=dotcom%3Afrontend-all&build=2308&stage=CODE&updateStrategy=MostlyHarmless&action=deploy) - [Deploy parts of build 2308 to CODE by previewing it first](https://riffraff.gutools.co.uk/preview/yaml?project=dotcom%3Afrontend-all&build=2308&stage=CODE&updateStrategy=MostlyHarmless) - [What's on CODE right now?](https://riffraff.gutools.co.uk/deployment/history?projectName=dotcom%3Afrontend-all&stage=CODE)

From guardian/actions-riff-raff.

prout-bot commented 5 days ago

Overdue on FRONTS-PROD, ADMIN-PROD (merged by @sookburt 30 minutes and 5 seconds ago) What's gone wrong?

prout-bot commented 5 days ago

Seen on FRONTS-PROD, ADMIN-PROD (merged by @sookburt 1 hour, 4 minutes and 37 seconds ago)