Skip to content

Commit 09d9c19

Browse files
committed
Remove unused enhanced json endpoints
1 parent 217ad6d commit 09d9c19

File tree

4 files changed

+2
-59
lines changed

4 files changed

+2
-59
lines changed

dotcom-rendering/src/server/handler.article.web.ts

Lines changed: 0 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -20,22 +20,6 @@ export const handleArticle: RequestHandler = ({ body }, res) => {
2020
res.status(200).set('Link', makePrefetchHeader(prefetchScripts)).send(html);
2121
};
2222

23-
export const handleArticleJson: RequestHandler = ({ body }, res) => {
24-
recordTypeAndPlatform('article', 'json');
25-
26-
const frontendData = validateAsFEArticle(body);
27-
const article = enhanceArticleType(frontendData, 'Web');
28-
const resp = {
29-
data: {
30-
// TODO: We should rename this to 'article' or 'FEArticle', but first we need to investigate
31-
// where/if this is used.
32-
CAPIArticle: article,
33-
},
34-
};
35-
36-
res.status(200).send(resp);
37-
};
38-
3923
export const handleInteractive: RequestHandler = ({ body }, res) => {
4024
recordTypeAndPlatform('interactive', 'web');
4125

dotcom-rendering/src/server/handler.front.web.ts

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -145,10 +145,6 @@ export const handleFront: RequestHandler = ({ body }, res) => {
145145
res.status(200).set('Link', makePrefetchHeader(prefetchScripts)).send(html);
146146
};
147147

148-
export const handleFrontJson: RequestHandler = ({ body }, res) => {
149-
res.json(enhanceFront(body));
150-
};
151-
152148
export const handleTagPage: RequestHandler = ({ body }, res) => {
153149
recordTypeAndPlatform('tagPage');
154150
const tagPage = enhanceTagPage(body);
@@ -157,7 +153,3 @@ export const handleTagPage: RequestHandler = ({ body }, res) => {
157153
});
158154
res.status(200).set('Link', makePrefetchHeader(prefetchScripts)).send(html);
159155
};
160-
161-
export const handleTagPageJson: RequestHandler = ({ body }, res) => {
162-
res.json(enhanceTagPage(body));
163-
};

dotcom-rendering/src/server/server.dev.ts

Lines changed: 1 addition & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -10,17 +10,11 @@ import {
1010
} from './handler.article.apps';
1111
import {
1212
handleArticle,
13-
handleArticleJson,
1413
handleBlocks,
1514
handleInteractive,
1615
} from './handler.article.web';
1716
import { handleEditionsCrossword } from './handler.editionsCrossword';
18-
import {
19-
handleFront,
20-
handleFrontJson,
21-
handleTagPage,
22-
handleTagPageJson,
23-
} from './handler.front.web';
17+
import { handleFront, handleTagPage } from './handler.front.web';
2418
import {
2519
handleCricketMatchPage,
2620
handleFootballMatchListPage,
@@ -97,15 +91,12 @@ const renderer = Router();
9791
// URL if req.params.url is present
9892
renderer.use(getContentFromURLMiddleware);
9993
renderer.get('/Article/*url', handleArticle);
100-
renderer.get('/ArticleJson/*url', handleArticleJson);
10194
renderer.get('/AMPArticle/*url', handleAMPArticle);
10295
renderer.get('/Interactive/*url', handleInteractive);
10396
renderer.get('/AMPInteractive/*url', handleAMPArticle);
10497
renderer.get('/Blocks/*url', handleBlocks);
10598
renderer.get('/Front/*url', handleFront);
106-
renderer.get('/FrontJSON/*url', handleFrontJson);
10799
renderer.get('/TagPage/*url', handleTagPage);
108-
renderer.get('/TagPageJSON/*url', handleTagPageJson);
109100
renderer.get('/EmailNewsletters/*url', handleAllEditorialNewslettersPage);
110101
renderer.get('/AppsArticle/*url', handleAppsArticle);
111102
renderer.get('/AppsInteractive/*url', handleAppsInteractive);
@@ -117,15 +108,12 @@ renderer.get('/CricketMatchPage/*url', handleCricketMatchPage);
117108
renderer.get('/FootballMatchSummaryPage/*url', handleFootballMatchPage);
118109
// POST routes for running frontend locally
119110
renderer.post('/Article', handleArticle);
120-
renderer.post('/ArticleJson', handleArticleJson);
121111
renderer.post('/AMPArticle', handleAMPArticle);
122112
renderer.post('/Interactive', handleInteractive);
123113
renderer.post('/AMPInteractive', handleAMPArticle);
124114
renderer.post('/Blocks', handleBlocks);
125115
renderer.post('/Front', handleFront);
126-
renderer.post('/FrontJSON', handleFrontJson);
127116
renderer.post('/TagPage', handleTagPage);
128-
renderer.post('/TagPageJSON', handleTagPageJson);
129117
renderer.post('/EmailNewsletters', handleAllEditorialNewslettersPage);
130118
renderer.post('/AppsArticle', handleAppsArticle);
131119
renderer.post('/AppsInteractive', handleAppsInteractive);

dotcom-rendering/src/server/server.prod.ts

Lines changed: 1 addition & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -12,17 +12,11 @@ import {
1212
} from './handler.article.apps';
1313
import {
1414
handleArticle,
15-
handleArticleJson,
1615
handleBlocks,
1716
handleInteractive,
1817
} from './handler.article.web';
1918
import { handleEditionsCrossword } from './handler.editionsCrossword';
20-
import {
21-
handleFront,
22-
handleFrontJson,
23-
handleTagPage,
24-
handleTagPageJson,
25-
} from './handler.front.web';
19+
import { handleFront, handleTagPage } from './handler.front.web';
2620
import {
2721
handleCricketMatchPage,
2822
handleFootballMatchListPage,
@@ -71,9 +65,7 @@ export const prodServer = (): void => {
7165
app.post('/AMPInteractive', logRenderTime, handleAMPArticle);
7266
app.post('/Blocks', logRenderTime, handleBlocks);
7367
app.post('/Front', logRenderTime, handleFront);
74-
app.post('/FrontJSON', logRenderTime, handleFrontJson);
7568
app.post('/TagPage', logRenderTime, handleTagPage);
76-
app.post('/TagPageJSON', logRenderTime, handleTagPageJson);
7769
app.post(
7870
'/FootballMatchListPage',
7971
logRenderTime,
@@ -104,7 +96,6 @@ export const prodServer = (): void => {
10496
getContentFromURLMiddleware,
10597
handleArticle,
10698
);
107-
app.use('/ArticleJson/*url', handleArticleJson);
10899

109100
app.get(
110101
'/AMPArticle/*url',
@@ -119,25 +110,13 @@ export const prodServer = (): void => {
119110
getContentFromURLMiddleware,
120111
handleFront,
121112
);
122-
app.get(
123-
'/FrontJSON/*url',
124-
logRenderTime,
125-
getContentFromURLMiddleware,
126-
handleFrontJson,
127-
);
128113

129114
app.get(
130115
'/TagPage/*url',
131116
logRenderTime,
132117
getContentFromURLMiddleware,
133118
handleTagPage,
134119
);
135-
app.get(
136-
'/TagPageJSON/*url',
137-
logRenderTime,
138-
getContentFromURLMiddleware,
139-
handleTagPageJson,
140-
);
141120

142121
app.get(
143122
'/EmailNewsletters',

0 commit comments

Comments
 (0)