Skip to content
Draft
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
35 commits
Select commit Hold shift + click to select a range
d792eb9
feat: add identity headers to support banner requests when user is si…
Mar 3, 2026
757303d
Merge branch 'main' into jm/feat-mparticle-headers
juabara Mar 3, 2026
7c3ea21
feat: use getAuthHeaders for support banner requests to include ident…
Mar 3, 2026
0d25675
Merge branch 'jm/feat-mparticle-headers' of github.com:guardian/dotco…
Mar 3, 2026
f92b557
Merge branch 'main' into jm/feat-mparticle-headers
juabara Mar 3, 2026
ae51860
Merge branch 'main' into jm/feat-mparticle-headers
juabara Mar 4, 2026
1c45443
Merge branch 'main' into jm/feat-mparticle-headers
juabara Mar 4, 2026
032f324
feat: mock contributions API in storybook to prevent hanging on conse…
Mar 4, 2026
a0a25b5
chore: remove commented-out alias assignment in Storybook config
Mar 4, 2026
d19eeee
Merge branch 'main' of github.com:guardian/dotcom-rendering into jm/f…
Mar 5, 2026
5dda22e
fix: webpack server build failing due broken parse5 source maps insid…
Mar 5, 2026
42c7887
refactor: update TopBarSupport to use `useAuthStatus`, add a timeout …
Mar 5, 2026
08cbf93
chore: fix empty line lint rule
Mar 5, 2026
854d94b
chore: better naming header variables
Mar 5, 2026
07a6f71
Merge branch 'main' into jm/feat-mparticle-headers
juabara Mar 5, 2026
2c89d8e
fix: missing headers in DecideLayout, Gallery and Sport Data storybooks
Mar 5, 2026
8832f4d
Merge branch 'main' into jm/feat-mparticle-headers
juabara Mar 5, 2026
4d11607
feat: use `useIsSignedIn` hook to determine user sign-in status.
Mar 5, 2026
c0a20e5
Merge branch 'main' into jm/feat-mparticle-headers
juabara Mar 5, 2026
f1d4c66
Slimify all sections and reduce font size in AB test
domlander Mar 5, 2026
b84c8b8
Temporarily comment out an AB test requirement for testing
domlander Mar 5, 2026
aab700a
Merge branch 'main' into jm/feat-mparticle-headers
juabara Mar 6, 2026
8f2ad68
Put sublinks in correct place for AB test
domlander Mar 6, 2026
5d70402
WIP
tomrf1 Mar 6, 2026
e0928e6
Labs palette updates for front section (#15491)
cemms1 Mar 9, 2026
f0befca
Merge branch 'main' into jm/feat-mparticle-headers
juabara Mar 10, 2026
befef82
Merge pull request #15462 from guardian/jm/feat-mparticle-headers
juabara Mar 10, 2026
250aba0
Merge branch 'tf-header-auth' of github.com:guardian/dotcom-rendering…
Mar 10, 2026
533159b
Merge pull request #15487 from guardian/doml/slim-homepage-ab-test-sl…
domlander Mar 10, 2026
76a21a6
Braze Banners Wrapper Design improvements (#15508)
andresilva-guardian Mar 10, 2026
65c071e
refactor: remove storybook contributions mock and simplify TopBarSupp…
Mar 10, 2026
c859d2c
Merge branch 'main' into jm/feat-mparticle-headers
juabara Mar 10, 2026
aa0b902
Merge pull request #15510 from guardian/jm/feat-mparticle-headers
juabara Mar 10, 2026
b079e0b
Remove deprecated 'fixed' containers
domlander Mar 9, 2026
db22948
Remove 'show more' functionality from containers
domlander Mar 11, 2026
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions ab-testing/config/abTests.ts
Original file line number Diff line number Diff line change
Expand Up @@ -109,14 +109,14 @@ const ABTests: ABTest[] = [
{
name: "fronts-and-curation-slim-homepage",
description:
"Test placing the Most Viewed and Deeply Read components in the right-hand column on the homepage.",
"Test slimming content and placing Most Popular components on the right-hand side on the UK front.",
owners: ["fronts.and.curation@guardian.co.uk"],
status: "ON",
expirationDate: `2026-04-28`,
expirationDate: "2026-04-28",
type: "server",
audienceSize: 0 / 100,
audienceSpace: "A",
groups: ["control", "variant"],
groups: ["control", "variant-one", "variant-two"],
shouldForceMetricsCollection: false,
},
{
Expand Down
47 changes: 26 additions & 21 deletions dotcom-rendering/.storybook/main.ts
Original file line number Diff line number Diff line change
Expand Up @@ -103,24 +103,34 @@ const webpackConfig = (config: Configuration) => {
const rules = config.module?.rules ?? [];

config.resolve ??= {};
config.resolve.alias ??= {};

// Mock JSDOM for storybook - it relies on native node.js packages
// Allows us to use enhancers in stories for better testing of components & full articles
config.resolve.alias['jsdom$'] = path.resolve(
__dirname,
'./mocks/jsdom.ts',
);
config.resolve.alias = {
...config.resolve?.alias,

// log4js tries to call "fs" in storybook -- we can ignore it
config.resolve.alias[
`${path.resolve(__dirname, '../src/server/lib/logging')}$`
] = path.resolve(__dirname, './mocks/log4js.ts');
Buffer: 'buffer',
react: 'react',
'react-dom': 'react-dom',

// Mock BridgetApi for storybook
config.resolve.alias[
`${path.resolve(__dirname, '../src/lib/bridgetApi')}$`
] = path.resolve(__dirname, './mocks/bridgetApi.ts');
// Mock JSDOM for storybook - it relies on native node.js packages
// Allows us to use enhancers in stories for better testing of components & full articles
jsdom$: path.resolve(__dirname, './mocks/jsdom.ts'),

// log4js tries to call "fs" in storybook -- we can ignore it
[`${path.resolve(__dirname, '../src/server/lib/logging')}$`]:
path.resolve(__dirname, './mocks/log4js.ts'),

// Mock BridgetApi for storybook
[`${path.resolve(__dirname, '../src/lib/bridgetApi')}$`]: path.resolve(
__dirname,
'./mocks/bridgetApi.ts',
),

// Mock identity auth frontend to prevent Storybook components from hanging in Pending
'@guardian/identity-auth-frontend': path.resolve(
__dirname,
'./mocks/identityAuthFrontend.ts',
),
};

const webpackLoaders = getLoaders('client.web');

Expand Down Expand Up @@ -149,12 +159,7 @@ const webpackConfig = (config: Configuration) => {
config.resolve.modules = [
...((config && config.resolve && config.resolve.modules) || []),
];
config.resolve.alias = {
...config.resolve.alias,
Buffer: 'buffer',
react: 'react',
'react-dom': 'react-dom',
};

return config;
};

Expand Down
8 changes: 8 additions & 0 deletions dotcom-rendering/.storybook/mocks/identityAuthFrontend.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
export const getIdentityAuth = () => ({
isSignedInWithAuthState: () =>
Promise.resolve({
isAuthenticated: false,
accessToken: undefined,
idToken: undefined,
}),
});
48 changes: 24 additions & 24 deletions dotcom-rendering/fixtures/manual/frontCollections.ts
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ const defaultGrouped = {

const defaultValues = {
backfill: [],
collectionType: 'fixed/small/slow-IV',
collectionType: 'static/medium/4',
config: {
showDateHeader: false,
},
Expand All @@ -35,13 +35,13 @@ export const testCollectionsUk = [
},
{
...defaultValues,
collectionType: 'fixed/small/slow-IV',
collectionType: 'static/medium/4',
containerPalette: 'LongRunningAltPalette',
displayName: 'Ukraine invasion',
},
{
...defaultValues,
collectionType: 'fixed/small/slow-V-mpu',
collectionType: 'static/medium/4',
displayName: 'News extra',
},
{
Expand Down Expand Up @@ -71,12 +71,12 @@ export const testCollectionsUk = [
},
{
...defaultValues,
collectionType: 'fixed/medium/slow-VI',
collectionType: 'flexible/general',
displayName: 'Lifestyle',
},
{
...defaultValues,
collectionType: 'fixed/medium/slow-VII',
collectionType: 'flexible/general',
displayName: 'Culture',
},
{
Expand All @@ -91,7 +91,7 @@ export const testCollectionsUk = [
},
{
...defaultValues,
collectionType: 'fixed/small/slow-IV',
collectionType: 'static/medium/4',
displayName: 'The rural network',
},
{
Expand All @@ -111,23 +111,23 @@ export const testCollectionsUk = [
},
{
...defaultValues,
collectionType: 'fixed/medium/slow-VI',
collectionType: 'flexible/general',
displayName: 'Multimedia',
},
{
...defaultValues,
collectionType: 'fixed/small/slow-IV',
collectionType: 'static/medium/4',
containerPalette: 'Branded',
displayName: 'Guardian Labs',
},
{
...defaultValues,
collectionType: 'fixed/medium/slow-XII-mpu',
collectionType: 'flexible/general',
displayName: 'Explore',
},
{
...defaultValues,
collectionType: 'fixed/small/slow-I',
collectionType: 'flexible/general',
displayName: 'The big picture',
},
{
Expand Down Expand Up @@ -162,7 +162,7 @@ export const testCollectionsUs = [
},
{
...defaultValues,
collectionType: 'fixed/small/slow-IV',
collectionType: 'static/medium/4',
displayName: 'In depth',
},
{
Expand All @@ -172,13 +172,13 @@ export const testCollectionsUs = [
},
{
...defaultValues,
collectionType: 'fixed/small/slow-IV',
collectionType: 'static/medium/4',
containerPalette: 'LongRunningAltPalette',
displayName: 'Ukraine invasion',
},
{
...defaultValues,
collectionType: 'fixed/medium/slow-VI',
collectionType: 'flexible/general',
displayName: 'Opinion',
},
{
Expand All @@ -193,7 +193,7 @@ export const testCollectionsUs = [
},
{
...defaultValues,
collectionType: 'fixed/small/slow-IV',
collectionType: 'static/medium/4',
displayName: 'Climate crisis',
},
{
Expand All @@ -218,7 +218,7 @@ export const testCollectionsUs = [
},
{
...defaultValues,
collectionType: 'fixed/medium/slow-VII',
collectionType: 'flexible/general',
displayName: 'Podcasts',
},
{
Expand All @@ -228,28 +228,28 @@ export const testCollectionsUs = [
},
{
...defaultValues,
collectionType: 'fixed/medium/slow-VI',
collectionType: 'flexible/general',
displayName: 'Culture',
},
{
...defaultValues,
collectionType: 'fixed/small/slow-IV',
collectionType: 'static/medium/4',
containerPalette: 'Branded',
displayName: 'Business briefs',
},
{
...defaultValues,
collectionType: 'fixed/medium/slow-VI',
collectionType: 'flexible/general',
displayName: 'Lifestyle',
},
{
...defaultValues,
collectionType: 'fixed/small/slow-IV',
collectionType: 'static/medium/4',
displayName: 'Take part',
},
{
...defaultValues,
collectionType: 'fixed/medium/slow-VI',
collectionType: 'flexible/general',
displayName: 'Explore',
},
{
Expand All @@ -259,7 +259,7 @@ export const testCollectionsUs = [
},
{
...defaultValues,
collectionType: 'fixed/medium/slow-VI',
collectionType: 'flexible/general',
displayName: 'In pictures',
},
{
Expand All @@ -282,7 +282,7 @@ export const brandedTestCollections = [
},
{
...defaultValues,
collectionType: 'fixed/small/slow-IV',
collectionType: 'static/medium/4',
containerPalette: 'Branded',
displayName: 'Guardian Labs',
},
Expand All @@ -293,7 +293,7 @@ export const brandedTestCollections = [
},
{
...defaultValues,
collectionType: 'fixed/small/slow-IV',
collectionType: 'static/medium/4',
containerPalette: 'Branded',
displayName: 'Guardian Labs',
},
Expand All @@ -304,7 +304,7 @@ export const brandedTestCollections = [
},
{
...defaultValues,
collectionType: 'fixed/small/slow-IV',
collectionType: 'static/medium/4',
containerPalette: 'Branded',
displayName: 'Guardian Labs',
},
Expand Down
Loading
Loading