fix: ci-2159 remove html-to-text package #56
Conversation
| }); | ||
| expect(result.description).to.equal("Hello I'm HTML!"); | ||
| }); | ||
| // TODO: https://financialtimes.atlassian.net/browse/CI-2226 reinstate summary |
There was a problem hiding this comment.
@debugwand this should not needed anymore with cp-pipeline. It relates to the part here
| // TODO: https://financialtimes.atlassian.net/browse/CI-2226 reinstate summary |
We can assume that will eventually be implemented also in FT-App in the same way. Please close the ticket, thanks
There was a problem hiding this comment.
is that the bit that updates what is visible in the page though? this is for the metadata in JSON LD
Currently
it no longer has the metadata about the links that would look like this, that goes after the end of the entry about the image
"liveBlogUpdate":[{"@type":"BlogPosting","headline":"US LNG pioneer Souki leaves top job at Tellurian","datePublished":"2023-12-08T23:40:41.160Z","dateModified":"2023-12-08T23:40:41.160Z"},{"@type":"BlogPosting","headline":"US stocks hit 20-month high as S&P 500 notches longest weekly win streak in 4 years","datePublished":"2023-12-08T21:38:21.810Z","dateModified":"2023-12-08T21:38:21.810Z"},{"@type":"BlogPosting","headline":"US vetoes UN resolution demanding Gaza ceasefire","datePublished":"2023-12-08T21:16:24.252Z","dateModified":"2023-12-08T21:16:24.252Z"},{"@type":"BlogPosting","headline":"Treasuries sell off after strong jobs report surprises traders","datePublished":"2023-12-08T19:16:06.751Z","dateModified":"2023-12-08T19:16:06.751Z"}, etc.etc.
(full paste of old and current outputs are in https://financialtimes.atlassian.net/browse/CI-2226)
No description provided.