-
Notifications
You must be signed in to change notification settings - Fork 280
Remove Express + unused dependencies #14028
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
Changes from 35 commits
97476b1
a53d505
6d0c67a
073ad43
832859e
248fac4
73c26bb
ac6b7e3
5feaf73
dd834df
56852e1
cd34e2e
ba66df3
e390dc7
126936f
8876548
f91d5e4
ef9245d
1b50a8d
0d27c94
3ffdace
4ebfe90
af520c0
6c7edbe
af9bff4
8731808
c6d0418
af9b799
e295133
82faa4d
819de9c
2bd84af
aeb4220
1a6291f
5d43b3f
d7d8fb1
909d4d4
1fb22ac
44898c7
9ddc9a2
50ac0b9
4ea2f3c
c149aab
4bf1a28
5c7d777
678d9e5
2d45d52
00426e5
f58a7d1
ed36699
2ddf519
b45fe41
3ea5238
1c75196
832823c
58318f8
4731e5a
ee8bbc2
fa33092
d219210
e0f7f21
9074f14
a798e3b
1868ad9
7333bd9
2e28576
6556fa8
26354a2
f206da8
6d51e33
341c9fa
35dce5d
7b5373c
c43cae6
7962a80
9e99a7b
acfb25b
a52f274
55bc4aa
aef56c5
74684f4
ef2910a
2216a64
0d3e3fd
b6286df
48e1287
68e3aa7
a5b1ad8
387b2cb
08a00af
3adabc3
473022c
6cbf90e
fa9379d
ea370fe
e6adffd
b5ee850
003d6a2
a66d901
eefaf22
f031c7e
7a29d48
80c19b6
bfc16dc
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||
|---|---|---|---|---|
|
|
@@ -11,8 +11,8 @@ import type { StorybookConfig } from '@storybook/react-webpack5'; | |||
| import alias from '../dirAlias'; | ||||
|
|
||||
| import { fontInfo } from '../src/app/components/ThemeProvider/fontFaces'; | ||||
|
|
||||
| const require = createRequire(import.meta.url); | ||||
| const MomentTimezoneInclude = require('../src/app/legacy/psammead/moment-timezone-include/src'); | ||||
| const DOT_ENV_CONFIG = dotenv.config({ quiet: true }); | ||||
|
|
||||
| const storybookConfig: StorybookConfig = { | ||||
|
|
@@ -116,6 +116,7 @@ const storybookConfig: StorybookConfig = { | |||
| new webpack.ProvidePlugin({ | ||||
| process: 'process/browser', | ||||
| }), | ||||
| new MomentTimezoneInclude({ startYear: 2010, endYear: 2026 }), | ||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Did we ever address the
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Next has it set correctly: simorgh/ws-nextjs-app/next.config.js Line 88 in e076421
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. shouldn't it be further in the future like 2028 or something?
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It can be, but that will generate more code: https://momentjs.com/timezone/docs/#:~:text=%2F%2F%20To%20keep%20all%20zones%20but%20limit%20data%20to%20specific%20years%2C%20use%20the%20year%20range%20options%20new%20MomentTimezoneDataPlugin%28%7B%20startYear%3A%20currentYear%20%2D%205%2C%20endYear%3A%20currentYear%20%2B%205%2C%20%7D%29%2C so its a bit of tradeoff between maintenance and some performance.
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I guess what we need to put a reminder in or we'll get a new years problem? |
||||
| ); | ||||
|
|
||||
| config.resolve!.fallback = { | ||||
|
|
||||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,7 +1,3 @@ | ||
| # Debugging in VS Code | ||
|
|
||
| The setup here mirrors the [`webpack:dev:server`](https://github.com/bbc/simorgh/blob/2944b327aac6fa22a8b5474d24e48fb631728431/package.json#L65) `package.json` script. You run it in the debug tab choosing 'Debug Express App'. | ||
|
|
||
|  | ||
|
|
||
| You can debug the NextJS app by choosing 'Debug Next.js: server-side'; this command has been adapted from the docs [here](https://nextjs.org/docs/pages/building-your-application/configuring/debugging#debugging-with-vs-code) |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -21,7 +21,7 @@ Afrique, Arabic, Azeri, Bengali, Gahuza, Hausa, Hindi, Indonesia, Japanese, Kyrg | |
|
|
||
| Given you are using Chrome browser. | ||
|
|
||
| 1. Download [ModHeader Chrome extension](https://www.google.com/search?q=modheader+chrome+pass+boolean\&rlz=1C5CHFA_enGB762GB762\&oq=modheader\&aqs=chrome.0.69i59l2j69i57j69i60j69i61j69i60j69i65l2.1039j0j7\&sourceid=chrome\&ie=UTF-8) | ||
| 1. Download [ModHeader Chrome extension](https://www.google.com/search?q=modheader+chrome+pass+boolean&rlz=1C5CHFA_enGB762GB762&oq=modheader&aqs=chrome.0.69i59l2j69i57j69i60j69i61j69i60j69i65l2.1039j0j7&sourceid=chrome&ie=UTF-8) | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. interesting it's removed the back slashes here
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Think this might have been Copilot |
||
|
|
||
| 2. With the ModHeader extension, add a header of `BBC-Adverts` with the value `true`. | ||
|
|
||
|
|
@@ -55,7 +55,7 @@ If you are unable to set a proxy on one of the Windows a11y laptops, you won't b | |
|
|
||
| e.g. `SIMORGH_PUBLIC_STATIC_ASSETS_ORIGIN=http://192.168.0.50:7080` | ||
|
|
||
| 3. Disable the `upgrade-insecure-requests` directive from the [cspHeader](https://github.com/bbc/simorgh/blob/latest/src/server/utilities/cspHeader/index.js#L419) | ||
| 3. Disable the `upgrade-insecure-requests` directive from the [cspHeader](https://github.com/bbc/simorgh/blob/latest/src/ws-nextjs-app/utilities/cspHeader/index.ts#L419) | ||
|
|
||
| 4. If you can't set the `BBC-Adverts` header in the Windows laptop, you will need to remove [this](https://github.com/bbc/simorgh/blob/latest/src/app/containers/Ad/Canonical/index.jsx#L63) condition from the codebase to be able to render the ad | ||
|
|
||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.