Merged
Conversation
barkertron
added a commit
that referenced
this pull request
Jun 3, 2024
* Nxt 180 scaffold pages (#131) * NXT-180 Add basic routing; refactor fonts * NXT-180 Add basic Storyblok integration * NXT-180 Display SB data on home and about pages * NXT-180 Fix TS issues, abstract Storyblok connection --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nxt 181 card components (#135) * NXT-181 Card grid homepage WIP * NXT-181 Fix TS errors * NXT-181 Add card component from NDS * NXT-181 TS integration for cards and homepage * NXT-181 Resolve Storyblok link objects * NXT-181 Fix links when SB returns empty link objects * NXT-181 Alter card grid layout, fix failing tests * NXT-181 Remove logging --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nxt 185 hero and catnav (#136) * NXT-185 Add hero and catnav components * NXT-185 Add SSL proxy for Storyblok preview mode * NXT-185 Resolve draft versions from context * NXT-185 Refactor page fetching, add catch-all about route * NXT-185 Breadcrumbs WIP --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-185 Add unit tests, fix broken page fetching * NXT-187 Refactor breadcrumb code (#137) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-188 corporate page metadata (#139) * NXT-188 Fix failing tests * NXT-188 Add metadata from Storyblok, update user-generated metadata * NXT-188 Add canonical and language link/meta tags --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Bump commit * Bump commit * Bump * Bump commit * Fix broken snapshot * Replace public config with server config for SB tokens * NXT-186 Static paths WIP (#153) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nxt 186 storyblok static paths (#155) * NXT-172 Add API endpoint for deleting fs cache entries (#122) * NXT-155 Change alert functionality to read from InDev * NXT-155 Updated snapshot test * NXT-155 Added alert logic to docs/consultation tabs for project page * NXT-155 Add API endpoint for deleting fs cache entries * NXT-155 Prevent cacheing of API cache deletion response * NXT-155 possible caching fixes (#134) * NXT-155 possible caching fixes * NXT-155 stop it generating multiple cache entries in different cases * NXT-155 move lowercase fixes to getCacheKey function * NXT-155 Lint cache api; send group keys in 400 err * NXT-155 fix tests --------- Co-authored-by: John Holland <john.holland@nice.org.uk> Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: Claire Hunter <claire.hunter@nice.org.uk> * NXT-178 Add international enquiry form and test (#132) * NXT-190 embedded doesnt exist for hub pages (#138) * NXT-186 Static paths WIP * NXT-186 Update linting rules --------- Co-authored-by: John Holland <john.holland@nice.org.uk> Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: Claire Hunter <claire.hunter@nice.org.uk> Co-authored-by: John Davey <66066685+johndavey72@users.noreply.github.com> * Nxt 186 storyblok static paths (#156) * NXT-172 Add API endpoint for deleting fs cache entries (#122) * NXT-155 Change alert functionality to read from InDev * NXT-155 Updated snapshot test * NXT-155 Added alert logic to docs/consultation tabs for project page * NXT-155 Add API endpoint for deleting fs cache entries * NXT-155 Prevent cacheing of API cache deletion response * NXT-155 possible caching fixes (#134) * NXT-155 possible caching fixes * NXT-155 stop it generating multiple cache entries in different cases * NXT-155 move lowercase fixes to getCacheKey function * NXT-155 Lint cache api; send group keys in 400 err * NXT-155 fix tests --------- Co-authored-by: John Holland <john.holland@nice.org.uk> Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: Claire Hunter <claire.hunter@nice.org.uk> * NXT-178 Add international enquiry form and test (#132) * NXT-190 embedded doesnt exist for hub pages (#138) * NXT-186 Static paths WIP * NXT-186 Update linting rules * NXT-186 Resolve custom rendering method --------- Co-authored-by: John Holland <john.holland@nice.org.uk> Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: Claire Hunter <claire.hunter@nice.org.uk> Co-authored-by: John Davey <66066685+johndavey72@users.noreply.github.com> * Add verbose logging to debug test env * Auto clear SB cache * NXT-211 Add simple server-side catch-all route (#162) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-197 Add breadcrumbs (#164) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Temporarily circumvent SB cache * Temp page header component update * Update Storyblok types; ignore generated file with ESLint * Correct linting setup * Make grid gutter loose * Emit logs from status page check * Update status page logs * Replace test status page errors with warnings * Fix lockfile * Update tsconfig settings * Add new SB types * WIP info page & nested rich text component * Nxt 229 news article page (#174) * chore: test updating the storyblok types from npm script * feat: test adding image to the storyblok hero and test using button as link if ctaLink and ctatext returned from Storyblok * chore: add storyblok richtext renderer as dep * feat: add NewsArticle component to render news articles in app - Add News Article component with PageHeader and richtext renderer for content. - Update PageHeader to render formatted date from design Dale H * style: test adding basic layout styles from scss * style: update image css * feat: first pass at blockquote we will use in the richtext examples in news articles * feat: test adding blockquote to the richtext renderer resolver * feat: first pass at NewsLetterSignup using DS components * styles: temp update to blockquote component style * feat: first pass at adding route for news with catch all for news-blogs-and-podcasts (TEMP till slug sturcture agreed) * chore: remove console log * refactor: update NewsArticle component to start using DS components * feat: first pass adding BlogPost component to render blog pages (TODO: move to its own route under pages dir) * refactor: remove empty h2 tag * feat: flesh out news article and move to correct path - Updated the news article to show panels on sidebar - updated StoryblokPageHeader to include tags and date (temporary test until new PageHeader version completed) - Added first pass on StoryblokImage component to use Storyblok image service, adds support for webp and jpeg formats Dale H * feat: first pass on sidebar panels for related news and links - Added first pass StoryblokRelatedLink and StoryblokRelatedNewsLink components - Added Panels to sidebar on NewsArticle Page TODO: update spacing across news article and it's components TODO: add tests for components and pages Dale H * refactor: update related news and link components to use Link component from next-web * chore: remove console.logs * feat: first pass at updating breadcrumbs - Update logic on breadcrumb function - Add path on getBreadcrumbs call, using partly hard coded path. TODO: get path dynamically, within getStaticProps Dale H * refactor: test PageHeader alpha release in StoryblokPageHeader * styles: update featuredImage offset styles to use custom property, so we can update the variable value inside of media queries * feat(ts): udpate type for image and pass rest prop to handle passing of common image attributes * refactor: update newsarticle image to take width and height and maintain 16/9 aspect ratio * test: update image component test and add fallback image for when src is empty * refactor: hardcode Breadcrumbs in news article. Comment out breadcrumb logic from StoryblokPageHeader to test news article * fix: remove updatedBreadcrumb * feat: start richtextRenderer component * styles: remove max width from blockquote to span full width of container * fix: revert StoryblokPageHeader component for stories that use the StoryblokPageHeader blok * feat: first pass on NewsPageHeader to use on news, in-depth, podcast and blog pages * feat: update richtext and news article components - Updated types for richtext component - Add NewsPageHeader and updated Richtext component to NewsArticle page component Dale H * styles: update style to handle images contained within richtext content * feat: first pass at YoutubeEmbed handled by rich text renderer * refactor: update newspageheader to use date util from repo * feat: Update richtext component to render inline images as StoryblokImage component ` * fix: add workaround to handle inline quotes in the richtext field of news stories NOTE: currently the children of the inline quote doesn't render correctly when using the blockquote component in the nodeResolver of storyblok-rich-text-renderer. We bump into a few issues around rendering and correct typing. I've hardcoded the html and imported styles so we can render the correctly styled quote TODO: Update the blockquote component or richtext types to handle passing through richtext quote children to the blockquote component * fix: update richtext interface, revert to initial interface from previous commit * tests: add tests for breadcrumbs and pageheader meta for tag and published date * fix: update breadcrumb name and resources link panel * test: remove test for fetchPriority attribute * refactor: update the inner Link component to use href over to attribute * tests: add relatednewslink tests * tests: first pass adding youtube embed tests TODO: add a field in storyblok for title so we can add an accessible title to the video embed Dale H * tests: firstpass adding tests for StoryblokRichText component * tests: first pass adding tests for Storyblok NewsArticle component * styles: update cta button so inner text wraps * fix: update types for newsarticle, add test to check whether signup action is rendered * tests: update NewsArticle tests * tests: first pass test news article slug page * fix: remove unused imports from richtext component and tests * chore: update design system version for new pageheader * refactor: update the newsarticle component to take new page header over newspageheader. - Update NewsArticle.tsx to use design system PageHeader new version - Add hardcoded breadcrumb values and handle in NewsArticle to be passed to PageHeader component TODO: Possible refactor of NewsPageHeader to be a generic component that can be used on news index and slug pages TODO: Image spacing to be handled via useEffect and ref to image to dynamically update the css varaible for its spacing Dale H ` * chore: rename pageheaderfooter that contains date and tag * refactor: update StoryblokImage and NewsArticle for overlap Added refs to NewsArticle and StoryblokImage so we can target them for updating the custom property value for the image overlap between header and body * style: update spacing around pageheader elements * tests: add further coverage on NewsArticle component * fix: udpate relatedlink and relatednewslink to use resolveStoyblokLink util function * round the pixels down on the featuredImageOffset value * fix: append /m/ for webp source image * fix: add title to Youtube embed for screen reader use * chore: update storyblok types * fix: update youtube iframe embed source value, remove comments * NICE-170 latest from nice & homepage (#175) * NICE-170 Homepage WIP * NICE-170 Latest news WIP * NICE-170 List latest news articles * NICE-170 Featured story WIP * NICE-170 Implement homepage hero * NICE-170 Action banner WIP * NICE-170 Resolved link, promo box WIP * NICE-170 Fix broken links for story objects * NICE-170 Complete "latest from NICE" section * NICE-170 Revert tsconfig, add NewsStory types * NICE-170 Add unit tests for homepage news section & components * NICE-170 Render news grid image as link, fix snapshots * NICE-170 Use better semantics, fix snapshots * NICE-170 misc fixes post review --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nxt 229 updates and fixes (#177) * fix: add empty dependancy array to useEffect to run once on mount, resize event for image * fix: add missing slash on jpeg src * feat: first pass adding author blok * styles: round image for author image * refactor: bring in update to fetchStory function and add author blok * first pass onblog post with listed authors - rough draft * update author component, extract the authorlist from blog post * update interface for blogpost and remove the inlined authorlist * fix: remove trailing comma from tsconfig * refactor: move authorlist to its own component, test author component * add authorList component and extend tests * start moving newspage header and child component into own files * fix: rename blogpost and newsarticle component to keep naming convention for Storyblok blok components * refactor: update AuthorList and tests * tests: start blogpost component tests * styles: update richtext elements vertical spacing and blog post * tests: update blog post slug page tests * tests: update youtube embed tests to include updated title property * tests: add missing slash from returned image source in tests * tests: update snapshot tests related to pageheader updates * styles: remove need for nested grid on BlogPost component, spacing updates on articles and posts * tests: update snapshots related to pageheader updates * tests: add testid to pageheader to test id applied, after pageheader updates * style: add commented styles to take over pageheader secondsection styles in NDS * feat: first pass at StoryblokIframe component * spike: spike nextjs/third-parties and react-lite youtube embeds - Tested next third parties, works but difficulty testing - Tested react-lite-youtube-embed, works and tested. TODO: test accessibility of react-lite-yt-embed further and against standard iframe implementation. TODO: ensure component is typed correctly * chore: remove spiked nextjs/third-parties * fix: remove incomplete import from richtext styles causing no styles to load * feat: use react-lite-youtube-embed for YT embed component * fix: update params and cookie props on on LiteYoutubeEmbed Issue with Safari not autoplaying the video once the poster play button was clicked. Adding cookie=true and updating params to include autoplay=1 solves this issue. * styles: update action banner spacing on newsarticle * styles: first pass normalise spacing to use DS spacing values * styles: normalise blockquote spacing * styles: tidy duplicated spacing * fix: add lodash debounce to handleResize functions on blog and news page components * tests: update breadcrumb test on slug page, and remove expect from youtubeembed test * NXT-250 Refactor fetchstories utility function (#179) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-251 Refactor NewsCard component (#180) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nxt 229 update tests and mocks (#181) * update test utils mocks for blog post and news article * fix: update type of blogpost blok and update tests to use mock util data * fix: udpate type of slug pages for blog and news to pass correct blok types * fix: udpate types and mock for newsarticle component - Update blok prop type and revert back to types generated by Storyblok - remove previous override type for news article - Update StoryblokNewsArticle to use mock data from test utils Dale H * refactor: update authorlist to render author blok over stoyblok component, and add className prop to AuthorBlok * tests: update test to check for author data in blogpost component * tests: update slug page tests on news and blog page slugs, using mock data from utils * tests: add richtext mock data and update tests that use richtext * NICE-169 secondary promo (#182) * NICE-169 Promo sections WIP * NICE-169 Complete Promo Box & tests --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nice 222 news blog style fixes (#184) * styles: update horizontal spacing * styles: update targetting on Youtube embed when child of richtext component * styles: update spacing on featuredImage and page header * refactor: normalise grid layout between newArticle and blogPost, update featuredImage margins * styles: switch featuredImage to 21/9 between sm and md breakpoints * styles: udpate spacing between main content/sidebar, header and action banner * styles: update bottom margin on youtube embed in richtext * styles: add fallback styles for inset and set button hover cursor to pointer * styles: update author styles from crit * styles: remove pageheader bottom margin * NICE-164 Implement Storyblok action banner (#185) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Big snapshot update * NICE-226 draft info page (#190) * NICE-226 Draft info page template * NICE-226 Revert Space ID * NICE-226 Remove log --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Fix linting error * Fix stylelint errors * Style fixes (#191) * fix: update styles to remove BEM convention in modules where we can * fix: add ignore on selector-class-pattern for targeting page header classes from DS in News and Blog components * Stylelint config update * NICE-178 Build the news listing template (#183) * NICE-178 Retrieve list of news articles from storyblok - wip * NICE-178 Add pagination component - wip * NICE-178 Return object with stories and total pages from storyblok response * NICE-178 Add current page and pagination destinations * NICE-178 Calculate total pages for pagination * NICE-178 Redirect to news/articles/ if page is outside bounds * NICE-178 Destructure gss props, return props directly * NICE-178 Componentise news list pagination * NICE-178 Rename prop as they aren't optional * NICE-178 Unit test newsListPagination wrapper * NICE-178 Revert tsconfig to fix issue with module resolution * NICE-178 Unit testing wip. Add npm script so space ID can be set and exported from terminal * NICE-178 Unit tests * NICE-178 Add action banner * NICE-178 Checkout web package.json etc to avoid merge conflict * NICE-178 Add NewsGrid component to articles list layout * NICE-178 Repurpose NewsGrid as NewsList for list pages * NICE-178 Move return in fetchStories to match storyblok-main * NICE-178 Add isListItem variant for news list. Setup basic style and layout for newslist page * NICE-178 Add featured story, sort stories by content.date descending. Update type useage * NICE-178 Add newslist nav component * NICE-178 Add npm script for updating types via shell * NICE-178 Update types * NICE-178 Add SEO * NICE-178 Adjust mobile styles for newscard * NICE-178 Add unit tests. Update snapshots * NICE-178 Add news nav to news index page. Change getActiveLink to match url paths exactly * NICE-178 Fix unit tests * NICE-178 Remove logging line * NICE-178 Move components to News dir to declutter main dir * NICE-178 Update location of pagination component * NICE-178 Prettify * NICE-178 Add unit tests for isNewsListItem variant * NICE-178 Comment out noisy router test * NICE-178 Skip first story on first page as it's featured. Articles index unit test - wip * NICE-178 Remove logger lines * NICE-178 Add todo for gssp on articles index * NICE-178 Unit tests * NICE-178 unit test gssp * NICE-178 remove worthless unit test * NICE-178 Update unit tests. Adjust border thickness for NewsCard * NICE-178 fix nesting level of selectors picked up by linting rule * NICE-178 Move storyblok utils mocks to outer describe * NICE-178 Remove magic number from unit test * NICE-178 Remove unnecessary string conversion in query * NICE-178 Handle storyblok fetchStories error condition * NICE-178 Revised error message * NICE-231 homepage update (#193) * NICE-231 Add homepage spacing, update promo box transparency * NICE-231 Complete Spotlight component * NICE-231 Update spotlight, add new news types * NICE-231 Fix rendering of homepage news links * NICE-231 Update snapshots * NICE-231 Update spotlight component to handle podcasts * NICE-231 Update snapshots, temporarily remove articles listing page unit tests * NICE-231 Fix TS errors * NICE-231 Homepage style updates * NICE-231 Fix snapshots --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Fix linting errors * Fix Card Grid link resolution * Nice 215 blog listing build (#195) * test: udpate snapshot test on newsarticle after updating mock data * udpate news and blog pages to filter scheduled stories according to date field * start validate route/pagination work * NICE-215 validateRouteParams wip * update types around validateRouteParams for storyblok * NICE-215 Handle no stories, parameterise validateRouteParams items and defaults * NICE-215 Fix starts_with after debug * test redirect and update blog page - first pass testing path redirect for out of range pagination pages - update blog listing page with new validateRouteParams - update news article listing page with resolvedUrl Authors: John Davey Dale Hudson * NICE-215 Remove debugging, tidy up out of range function * NICE-215 unit test featured story prop in gssp * NICE-215 pass sbParams differently, clean up types * NICE-215 Handle author content in FeaturedStory component * NICE-215 Change friendlyDate from toLocaleDateString to Intl.DateTimeFormat as this is more performant * NICE-215: update styles and destructure props for author in featuredStory * NICE-215: add first pass at author in newscard * NICE-215: update where we set AuthorHeadingElement * NICE-215: fix no rendering of secondSection if no authors length * NICE-215(fix): layout of author block * update storyblokimage for better handling of serviceOptions * NICE-215: update author card with iscardauthor and headinglevel props - update author to handle different heading levels with default to h4 - add prop to handle author when in card, updated styles for this too * NICE-215: update featured card with updated author component * remove redundant function and logs from SB utils * NICE-215(fix): update how author is handled in blogpost now author component updated * NICE-215(styles): udpate author styles in news and featuredStory cards * NICE-215: update full_slug link urls to absolute urls * NICE-215: udpate perf of background image and author keys - Added Storyblok image service parameters to resize and reduce quality of images to improve image loading times - Changed id for author lists within news and featured story cards * NICE-215: revert jpeg sources ready to refactor to use all image service options * NICE-215: add iframe component to storyblok init * NICE-215(fix): update image to ternary for correct loading of fallback image * NICE-215: fix Youtube embed button aria - remove announce prop from youtube embed, was duplicating aria label values - add fireevent to button click event in YT embed test * NICE-215(refactor): Update StoryblokImage component - Update image component to handle Storyblok image service options - As first pass we only cater for service options width, height, smart, quality - Update components using the StoryblokImage component with updated serviceOptions so we can optimise images currently in use. Dale H * NICE-215: handle error for empty stories array --------- Co-authored-by: John Davey <john.davey@nice.org.uk> Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * NICE-238 Add podcast page (#194) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * return null if author is string (#196) * Nice 215 fix tests (#197) * NICE-215: fix background images using SB image service by encoding brackets () * NICE-215: remove author as string from newsArticle as not part of content model * NICE-215(fix): Update snapshots for link and bgimg - updated snapshots for updated absolute url link handling in news and featured cards - updated snapshots for encoded background image urls containing storyblok image service filters - updated snapshots for featured and news cards with added authors John Davey Dale Hudson * fix pagination tests with updated prop names * NICE-215(fix): update snapshots from image component updates * NICE-215(fix): udpate tests after AuthorList refactor * NICE-215(fix): handle blogpost authors when array empty Handle blog posts author when they are pubilshed with an unpublished or no author attached. John D Dale H * Nice 231 homepage update (#198) * NICE-231 Add homepage spacing, update promo box transparency * NICE-231 Complete Spotlight component * NICE-231 Update spotlight, add new news types * NICE-231 Fix rendering of homepage news links * NICE-231 Update snapshots * NICE-231 Update spotlight component to handle podcasts * NICE-231 Update snapshots, temporarily remove articles listing page unit tests * NICE-231 Fix TS errors * NICE-231 Homepage style updates * NICE-231 Fix snapshots * NICE-231 Resolve merge conflicts * NICE-231 Homepage UI updates * NCIE-231 Limit news card to 1 author * NICE-231 Update author UI, change absolute URLs * NICE-231 Fix snapshots * NICE-231 Update Promobox mock data and snapshot --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nice 217 podcast listing page (#199) * NICE-217: podcast listing page, initial setup - Condtionally render newscard image - Update newscard author, return null if author is string D Hudson J Davey * NICE-217 Update podcast listing page - Make newsCard image optional - Fix broken unit test by removing redundant console log - Early return if author is just a string - Make newsListNav propless with destinations contained as they won't change - First pass at podcast listing page Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NICE-217 Add unit tests to cover optional showImage Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NICE-217 Update lead text on podcast list page * NICE-217: update breadcrumbs with absolute links --------- Co-authored-by: John Davey <john.davey@nice.org.uk> Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NICE-248 Resolve author linting issues (#200) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NICE-218 First pass at in-depth listing page (#201) Co-authored-by: John Davey <john.davey@nice.org.uk> Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * Nxt 270 news homepage (#202) * NXT-270 News homepage WIP * NXT-270 More news homepage WIP * NXT-270 Complete news homepage --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Fix linting issue * Add remote pattern image config * NICE 253 Refactor init storyblok (#204) * proof that separating initStoryblok enables proper mocking * streamline component imports for storyblokcomponent pages Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> --------- Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NICE-253 Fix issue with initStoryblok refactor breaking the catch all slug. Optimise component object for lean bundle. (#207) Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NXT-269(fix): filter articles out of latest news with future date (#208) * NICE-254 Handle image optimisation (#205) * NICE-254 Optimise images coming from storyblok image service. Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NICE-254 Update snapshots * NICE-254 Add optimise image util and unit test * NICE-254 Reinstate storyblokhero --------- Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * Nxt 292 blog page fixes from feedback (#209) * NXT-292: fix blog post author and author list headings * NXT-292: fix headings, not to render when no content * Merge storyblok-main Co-authored-by: John Davey <johndavey72@users.noreply.github.com> --------- Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * Nxt 292 blog page fixes from feedback (#210) * NXT-292: fix blog post author and author list headings * NXT-292: fix headings, not to render when no content * Merge storyblok-main Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * NXT-292 linting * NXT-292 merge sbmain --------- Co-authored-by: John Davey <johndavey72@users.noreply.github.com> Co-authored-by: John Davey <john.davey@nice.org.uk> * NXT-292: update homepage snapshot * NICE-255 Change enablejsapi from 0 to 1 (#212) * NICE-250 Managing focus on pagination route changes (#214) * spike pagination announcement for screenreader users - Add a div calling out the current page and total pages when navigating via screenreader - Update focus to hidden div, just above NewsList - this is one option of a few we need to test, preferably with screenreader users. - Options to go through linked on ticket: NICE-250 * NICE-250 Use Announcer component for pagination changes * NICE-250 add aria-hidden on focus element to prevent screen reader reading out 'blank' * NICE-250 Add announcer to other list pages. Pagination focus hook wip Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NICE-250 Roll out pagination focus to other list pages. Add component for pagination focus. * NICE-250 Remove console log * NICE-250 Add unit tests for pagination focus component * NICE-250 Refactor announcer code into a re-useable component * NICE-250 Remove unused deps and commented out code --------- Co-authored-by: dalecreativ <dalefrontend@gmail.com> Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NICE-260 Calculate canonical urls for slug pages and unit test (#216) Co-authored by Dale Hudson * Nxt 283 font size fixes (#215) * NXT-283: Update root font size to handle narrow viewport sizes. Co-authored by: John Davey * NXT-283: update font sizes for h1 on smaller screen sizes Targetting hero and page headings for home, news article, and blog post pages. Co-authored by: John Davey * NXT-283 Adjust h1 font-size for narrow viewport. Apply word-break and hyphenation for very narrow viewports Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> --------- Co-authored-by: John Davey <john.davey@nice.org.uk> Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * Nice 261 refactor image optimisation (#217) * NXT-261: refactor constructStoryBlokImageSrc to encodeParens and handle background images Co-authored by: John Davey. * NXT-261: null check image on podcast slig page * NXT-261: remove optimiseImage import from tests * NICE-261: update failing snapshot from encodeParens update (#218) * Nice 266 homepage spacing (#221) * NICE-266 Update homepage spacing; restore blog authors on news homepage * NICE-266 Miscellaneous QA fixes * NICE-266 Update blog page title * NICE-266 Set podcast image to unoptimised * NICE-266 Prevent future articles from appearing on news homepage * NICE-266 Pedantry --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nice 266 homepage spacing (#223) * NICE-266 Update homepage spacing; restore blog authors on news homepage * NICE-266 Miscellaneous QA fixes * NICE-266 Update blog page title * NICE-266 Set podcast image to unoptimised * NICE-266 Prevent future articles from appearing on news homepage * NICE-266 Pedantry * NICE-266 Set base URL for podcast image --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nice 266 homepage spacing (#224) * NICE-266 Update homepage spacing; restore blog authors on news homepage * NICE-266 Miscellaneous QA fixes * NICE-266 Update blog page title * NICE-266 Set podcast image to unoptimised * NICE-266 Prevent future articles from appearing on news homepage * NICE-266 Pedantry * NICE-266 Set base URL for podcast image * NICE-266 Pluralise podcasts label --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NICE-283 spacing again (#229) * NICE-283 Make spacing and content changes * NICE-283 Spacing, routing, text updates * NICE-283 A11y fixes * NICE-238 Amend news article action banner spacing * NICE-283 Update action banner spacing * NICE-283 Restore access token config --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Update snapshots * NXT-276 Update default image with base URL from config * NXT-217 Use Ocelot for Storyblok cacheing (#225) * NXT-217 Installed Ocelot.Administration NuGet package * NXT-217 Added Storyblok config to Ocelot routes * NXT-217 Configure Storyblok to use Ocelot * NXT-217 Add Ocelot Administration endpoint * NXT-217 Refactor Ocelot cache config settings * NXT-217 Add Ocelot for Storyblok section to readme * NXT-217 Update Readme --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NICE-291 Return 404 for root catch all unless env var enables it (#230) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-346 Revert env var config, add test page (#232) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nxt 346 revert env vars (#233) * NXT-346 Revert env var config, add test page * NXT-346 Add test env var to config --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * fix: update default ocelotEndpoint to empty value * Remove env var from next config * NXT-319, NICE-251 unit testing utility functions and news index pages, adding response status from SB API (#231) * initial commit * add boilerplate for get server side props * update validaterouteparams to return redirect if page is less than 1 TODO: alternative approach of assigning page to 1 if query.page is less than 1 * update mock data and refactor mocks for listing page * stub out getStoryblokAPi get method and modify response per test * Added json files for various responses Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * Added further test for fetchStory util co-authored by Dale Hudson * NXT-319 Add server error unit test Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NXT-319 Add unit tests for fetchStories co-authored by Dale Hudson * NXT-319 Add fetchLinks unit tests Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NXT-319 Add fetchLinks unit tests Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NXT-319 Remove promise.reject from fetchStories to prevent unreachable code issue. Tidy up imports in storyblok utils. Co-authored by Dale Hudson * NXT-319 Change mock links success response to match item returned from storyblok/react. Add note to investigate jest setup mocking issue * NXT-319 Add getBreadcrumbs unit tests * NXT-319 Add error handling to friendlyDate. Unit test friendlyDate * NXT-301: add test coverage for getNewsTypes * NXT-319: update validateRouteParams query.page to be set to one if less than 1 * NXT-319: start validateRouteParams test coverage * NXT-319: update news index page tests after updates * NXT-319: test for success and error responses from fetchStories returned to valiudateRouteParams * NXt-319: remove skip test validateRouteParam * NXT-319 Handle no stories on index pages. Return not founds for incorrect query page param. Refactor and add unit tests for validateRouteParams. Remove log lines. Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NXT-319 Remove unused spy. Add unit test todos for news articles listing" * NXT-319 Add unit test to check undefined return from getSlugFromParams. Remove unused dep * NXT-319 Add unit tests for news article list page. Refactor around thrown errors from validateRouteParams. Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NXT-319 Unit tests of news articles renderins. Gssp correct props unit test. * NXT-319 Rendering unit tests for news articles listing Co-authored by: Dale Hudson * NXT-319 Convert in-depth gssp to try catch format. Adjust types to separate error props from success props. * NXT-319 Move isError util function to storyblok utils and unit test. Make public facing error messages more generic". Update snapshot. * NXT-319 Update blog posts index gssp to try catch format. Separate error and success props and types" * NXT-319: update podcast gssp to handle errors from api * NXT-319 Remove resolved url from gssp args. Update unit tests to reflect removal of API details in error messages. * NXT-319 Add unit tests for news index pages * NXT-319 Add further unit tests for news index pages. * NXT-319 Refactor blog slug page. Added blog slug page unit tests. Added mock success response for blog post. * NXT-319 Refine types for fetchStory, blog slug page and unit test mock data types. Remove error property from fetchStories. Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NXT-319 Remove unused deps * NXT-319: remove test backup files * NXT-319: refactor news article slug with TODO comments for discussion * NXT-319: add test coverage for articles slug page * NXT-319: add TODO for discussion on fetchStory * NXT-319: add snapshot for news article page * NXT-319: remove redundant imports from news slug tests * NXT-319: add mockData for podcast page * NXT-319: refactor podcast slug page in line with other slug pages * NXT-319: add tests for podcast slug page with snapshot * NXT-319: udpate type for newsarticle fetch and props. Add logger on getadditionalMetaTag failure * NXT-319: add useMemo back to getAdditionalMetatags in blog post page * NXT-319: add logger to getAdditionalMetaTags in podcast page * NXT-319: remove unused imports from news slug page * Add further test coverage for podcast slug page - updated snapshots Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * NXT-319: remove redundant mocks from blog slug page tests * NXT-319 Error handling for news index and unit test - wip * NXT-319: test news homepage, gssp and error handling * NXT-319: add tests for homepage and handle fetch errors from API. Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * NXT-319: fix linting errors Co-authored by: John Davey * NXT-319: update error handling in utils. consistant messaging and var naming * NXT-319: remove TODOs * NXT-319: update homepage snapshot and test with error message update Co-authored by: John Davey * NXT-319: replace isError from utils with lodash isError equivalent Co-authored by: John Davey * NXT-319: remove isError import from test * NXT-319: move TODOs to tech debt and remove redundant comments. - update podcast listing mock data Co-authored by: John Davey --------- Co-authored-by: John Davey <john.davey@nice.org.uk> Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * Nxt 301 blog page fixes from feedback (#219) * NXT-301: order list of authors alphabetically * NXT-301: add border and spacing to author section in blog header * NXT-301: update author section spacing and remove bottom margin from any richtext component last child element * NXT-301: remove TODOs that have been resolved through updates or discussions * NXT-301: increase on author name if img is included * NXT-301: add role presentation to images carrying no alt text * NXT-301: update width of PageHeader fullwidth-container Full width container updates changed the alignment of pageheader to the rest of content on blog and news articles. Updated so content between header and main content is aligned. * NXT-301: add stylelint ignore rule for global styles targetting page-header * NXT-301: update author blok border with token for colour * NXT-301: revert page header container horizontal spacing. * NXT-301: pageheader author block spacing - Only onclude border and bottom padding over breakpoint * NXT-301: first pass at updating the richtext image aspect ratios based on storyblok image width and heights * NXT-301: update test data to have accurate url structure for SB CDN images * NXT-301: update displayed dates to use the time tag. - update components that display dates to use time tage - update the mock data for dates - refactor the news/blog header meta to use getNewsType util TODO: refactor tag and date into meta component for reuse across other components Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * NXT-301: update richtext image - update aspect ratio, width and height attrs using SB image url path to get width and height * NXT-301: replace storyblok cdn and link domain from url mock data. Mitigate calls to actual url. * NXT-301: upate storyblok-data mock urls * NXT-301: update snapshots after mock data changes to urls * NXT-301: update styles to nudge featured image - Nudge pixels on page header and featuredimage across news and blog page templates - Keeps margin top within 2rem of page header bottom content TODO: revisit handling of featuredImage offsets. Adding to tech debt for post go live Dale H * NXT-301: remove TODOs and put in tech debt log for discussion * NXT-301: move setComponent causing rendering issues --------- Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * fix: add back lint disable line for global targetting of css from DS * Main merge (#242) * NXT-246 Indicator Old URL mixed casing - redirects not working correctly (#192) * NXT-246 Added middleware to convert all resolved urls to lowercase Next js is routing to different pages/components if the url uses mixed casing compared to lowercase. This fixes that. * NXT-246 Fixed check to compare mapped indicator url with resolved url * NXT-246 Fixed linting issues * NXT-246 Added unit test for middleware function * NXT-330 Indicators change chapters to chapter (#226) * change chapters to chapter in indicators [slug] and update unit tests * NXT-330 Removed obsolete snapshots --------- Co-authored-by: John Davey <john.davey@nice.org.uk> * NXT-337 Fix for empty product panel (#227) * NXT-337 Fix for missing _embedded.niceIndevPanel Undefined when no documents are present. * NXT-337 Further null checks for panel list * NXT-343 Exclude next-js manifest files from url lowercasing middlewar… (#228) * NXT-343 Exclude next-js manifest files from url lowercasing middleware and unit test. Co-authored-by: John Holland * NXT-343 Modify comment to trigger TC build * NXT-353 Remove middleware page (#235) * NXT-353 Remove middleware page * NXT-353 Reinstate error causing middleware files * NXT-242 Add Terraform config (#176) * NXT-210 Add test logs to status page * NXT-210 Move logging test into forms dir * NXT-242 Add Terraform config * NXT-242 Update vars * NXt-242 Change ip to non public * NXT-241-new k6 scripts - js files and json files * NXT-242 Change bucket name * NXT-242 Add values into json * NXT-242 Update tfvars * NXT-242 Change to octo vars * NXT-242 Update tf vars * NXT-242 Fix typo * NXT-242 Fix issues * NXT-242 Update tfvars * NXT-241-updates to perfomance test k6 scripts * NXT-241-revert changes to vscode settings * NXT-241-remove typeahead terms from combined scripts and cleanup * NXT-241-added more paths to the combined json file * NXT-241-edited galp_pages test data * NXT-242 Added load balancer config to tf * NXT-242 Add build number to config dir * NXT-242 Added diagnostic info * NXT-242 Update efs volume path * NXT-242 Update var names * NXT-242 Add node env * NXT-242 Refactor TF * NXT-242 Revert config.yml * NXT-242 Update default.yml * NXT-242 Remove comment * NXT-242 Variablise CPU and Memory * NXT-242 Remove breaking characters * NXT-242 Add variable characters * NXT-242 Change variable type to number * NXT-242 Move pino-mqrc.json * NXT-242 Fix pino-mq config problem * NXT-242 Fix issues with pino-mq * NXT-288 Enable AWS Exec # Conflicts: # aws/terraform/terraform.tfvars # aws/terraform/variables.tf * NXT-242 Add server count variable, remove cloudwatch variable * NXT-241 Add basic test and change dir * NXT-241-fix-merge-commits * NXT-241-add-more-test-data * NXT-241-add-stages-plus-the-noConnectionReuse * NXT-241-prettier-fixes * NXT-242 Add teamcity aws config * NXT-242 Add extra stuff to k6 testing file * NXT-347 Update testing files * NXT-242 Add gitignore * NXT-242 Add readme.md * NXT-350 Add curl to Docker * NXT-350 Add healthcheck * NXT-242 Remove PM2 * NXT-242 Update readme * NXT-210 Update logtest.page.tsx * NXT-242 Remove performance testing * NXT-242 Apply TF format * NXT-242 Fix typo * NXT-242 Add comments to readme * NXT-242 Update readme --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: FunmiOsho <60607637+FunmiOsho@users.noreply.github.com> Co-authored-by: lizaadebowale <liza.adebowale@nice.org.uk> Co-authored-by: L-Vazquez <37586672+L-Vazquez@users.noreply.github.com> * Update package lock --------- Co-authored-by: John Holland <john.holland@nice.org.uk> Co-authored-by: John Davey <john.davey@nice.org.uk> Co-authored-by: John Davey <66066685+johndavey72@users.noreply.github.com> Co-authored-by: Colin M <12713935+colinmazu@users.noreply.github.com> Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: FunmiOsho <60607637+FunmiOsho@users.noreply.github.com> Co-authored-by: lizaadebowale <liza.adebowale@nice.org.uk> Co-authored-by: L-Vazquez <37586672+L-Vazquez@users.noreply.github.com> * Nice 249 sitemap generation (#245) * NICE-249 Fetch news content for sitemap from Links API, unignore sitemaps from Git, edit static sitemap from Orchard * NICE-249 Complete sitemap generation * NICE-249 Scrub auto-generated sitemap --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-345 Consolidate preview and public access tokens, remove env test page (#246) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-369 Add redirects for news section (#247) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Remove cv parameter from sbParams if Ocelot not present * Fix incorrect news redirect * Fix failing test due to CV parameter * Storyblok merge main (#250) * NXT-246 Indicator Old URL mixed casing - redirects not working correctly (#192) * NXT-246 Added middleware to convert all resolved urls to lowercase Next js is routing to different pages/components if the url uses mixed casing compared to lowercase. This fixes that. * NXT-246 Fixed check to compare mapped indicator url with resolved url * NXT-246 Fixed linting issues * NXT-246 Added unit test for middleware function * NXT-330 Indicators change chapters to chapter (#226) * change chapters to chapter in indicators [slug] and update unit tests * NXT-330 Removed obsolete snapshots --------- Co-authored-by: John Davey <john.davey@nice.org.uk> * NXT-337 Fix for empty product panel (#227) * NXT-337 Fix for missing _embedded.niceIndevPanel Undefined when no documents are present. * NXT-337 Further null checks for panel list * NXT-343 Exclude next-js manifest files from url lowercasing middlewar… (#228) * NXT-343 Exclude next-js manifest files from url lowercasing middleware and unit test. Co-authored-by: John Holland * NXT-343 Modify comment to trigger TC build * NXT-353 Remove middleware page (#235) * NXT-353 Remove middleware page * NXT-353 Reinstate error causing middleware files * NXT-242 Add Terraform config (#176) * NXT-210 Add test logs to status page * NXT-210 Move logging test into forms dir * NXT-242 Add Terraform config * NXT-242 Update vars * NXt-242 Change ip to non public * NXT-241-new k6 scripts - js files and json files * NXT-242 Change bucket name * NXT-242 Add values into json * NXT-242 Update tfvars * NXT-242 Change to octo vars * NXT-242 Update tf vars * NXT-242 Fix typo * NXT-242 Fix issues * NXT-242 Update tfvars * NXT-241-updates to perfomance test k6 scripts * NXT-241-revert changes to vscode settings * NXT-241-remove typeahead terms from combined scripts and cleanup * NXT-241-added more paths to the combined json file * NXT-241-edited galp_pages test data * NXT-242 Added load balancer config to tf * NXT-242 Add build number to config dir * NXT-242 Added diagnostic info * NXT-242 Update efs volume path * NXT-242 Update var names * NXT-242 Add node env * NXT-242 Refactor TF * NXT-242 Revert config.yml * NXT-242 Update default.yml * NXT-242 Remove comment * NXT-242 Variablise CPU and Memory * NXT-242 Remove breaking characters * NXT-242 Add variable characters * NXT-242 Change variable type to number * NXT-242 Move pino-mqrc.json * NXT-242 Fix pino-mq config problem * NXT-242 Fix issues with pino-mq * NXT-288 Enable AWS Exec # Conflicts: # aws/terraform/terraform.tfvars # aws/terraform/variables.tf * NXT-242 Add server count variable, remove cloudwatch variable * NXT-241 Add basic test and change dir * NXT-241-fix-merge-commits * NXT-241-add-more-test-data * NXT-241-add-stages-plus-the-noConnectionReuse * NXT-241-prettier-fixes * NXT-242 Add teamcity aws config * NXT-242 Add extra stuff to k6 testing file * NXT-347 Update testing files * NXT-242 Add gitignore * NXT-242 Add readme.md * NXT-350 Add curl to Docker * NXT-350 Add healthcheck * NXT-242 Remove PM2 * NXT-242 Update readme * NXT-210 Update logtest.page.tsx * NXT-242 Remove performance testing * NXT-242 Apply TF format * NXT-242 Fix typo * NXT-242 Add comments to readme * NXT-242 Update readme --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: FunmiOsho <60607637+FunmiOsho@users.noreply.github.com> Co-authored-by: lizaadebowale <liza.adebowale@nice.org.uk> Co-authored-by: L-Vazquez <37586672+L-Vazquez@users.noreply.github.com> * NXT-339 Update readme (#240) * NXT-353,NXT-343 404 errors on nextweb (#234) * NXT-343 Exclude next-js manifest files from url lowercasing middleware and unit test. Co-authored-by: John Holland * NXT-343 Modify comment to trigger TC build * NXT-343 Change url.pathname check to include _next/data and _next/static paths to avoid request errors * NXT-343 Add test for _next/data url condition * NXT-343 lowercase any mixed case indicators or guidance urls * NXT-343 Remove middleware page and unit tests * NXT-343 Hardcode indicators legacy redirects * NXT-343 Move indicators redirects from file directly to the config * NXT-343 Trying separate redirects file again * Revert "NXT-343 Trying separate redirects file again" This reverts commit f8d73685610fe09b773a9dd76bf3b26be8b8df01. * NXT-343 Correct redirects where there are duplicates with same case or missing lowercase version * Update package lock --------- Co-authored-by: John Holland <john.holland@nice.org.uk> Co-authored-by: John Davey <john.davey@nice.org.uk> Co-authored-by: John Davey <66066685+johndavey72@users.noreply.github.com> Co-authored-by: Colin M <12713935+colinmazu@users.noreply.github.com> Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: FunmiOsho <60607637+FunmiOsho@users.noreply.github.com> Co-authored-by: lizaadebowale <liza.adebowale@nice.org.uk> Co-authored-by: L-Vazquez <37586672+L-Vazquez@users.noreply.github.com> * Storyblok main merge conflicts (#252) * NXT-246 Indicator Old URL mixed casing - redirects not working correctly (#192) * NXT-246 Added middleware to convert all resolved urls to lowercase Next js is routing to different pages/components if the url uses mixed casing compared to lowercase. This fixes that. * NXT-246 Fixed check to compare mapped indicator url with resolved url * NXT-246 Fixed linting issues * NXT-246 Added unit test for middleware function * NXT-330 Indicators change chapters to chapter (#226) * change chapters to chapter in indicators [slug] and update unit tests * NXT-330 Removed obsolete snapshots --------- Co-authored-by: John Davey <john.davey@nice.org.uk> * NXT-337 Fix for empty product panel (#227) * NXT-337 Fix for missing _embedded.niceIndevPanel Undefined when no documents are present. * NXT-337 Further null checks for panel list * NXT-343 Exclude next-js manifest files from url lowercasing middlewar… (#228) * NXT-343 Exclude next-js manifest files from url lowercasing middleware and unit test. Co-authored-by: John Holland * NXT-343 Modify comment to trigger TC build * NXT-353 Remove middleware page (#235) * NXT-353 Remove middleware page * NXT-353 Reinstate error causing middleware files * NXT-242 Add Terraform config (#176) * NXT-210 Add test logs to status page * NXT-210 Move logging test into forms dir * NXT-242 Add Terraform config * NXT-242 Update vars * NXt-242 Change ip to non public * NXT-241-new k6 scripts - js files and json files * NXT-242 Change bucket name * NXT-242 Add values into json * NXT-242 Update tfvars * NXT-242 Change to octo vars * NXT-242 Update tf vars * NXT-242 Fix typo * NXT-242 Fix issues * NXT-242 Update tfvars * NXT-241-updates to perfomance test k6 scripts * NXT-241-revert changes to vscode settings * NXT-241-remove typeahead terms from combined scripts and cleanup * NXT-241-added more paths to the combined json file * NXT-241-edited galp_pages test data * NXT-242 Added load balancer config to tf * NXT-242 Add build number to config dir * NXT-242 Added diagnostic info * NXT-242 Update efs volume path * NXT-242 Update var names * NXT-242 Add node env * NXT-242 Refactor TF * NXT-242 Revert config.yml * NXT-242 Update default.yml * NXT-242 Remove comment * NXT-242 Variablise CPU and Memory * NXT-242 Remove breaking characters * NXT-242 Add variable characters * NXT-242 Change variable type to number * NXT-242 Move pino-mqrc.json * NXT-242 Fix pino-mq config problem * NXT-242 Fix issues with pino-mq * NXT-288 Enable AWS Exec # Conflicts: # aws/terraform/terraform.tfvars # aws/terraform/variables.tf * NXT-242 Add server count variable, remove cloudwatch variable * NXT-241 Add basic test and change dir * NXT-241-fix-merge-commits * NXT-241-add-more-test-data * NXT-241-add-stages-plus-the-noConnectionReuse * NXT-241-prettier-fixes * NXT-242 Add teamcity aws config * NXT-242 Add extra stuff to k6 testing file * NXT-347 Update testing files * NXT-242 Add gitignore * NXT-242 Add readme.md * NXT-350 Add curl to Docker * NXT-350 Add healthcheck * NXT-242 Remove PM2 * NXT-242 Update readme * NXT-210 Update logtest.page.tsx * NXT-242 Remove performance testing * NXT-242 Apply TF format * NXT-242 Fix typo * NXT-242 Add comments to readme * NXT-242 Update readme --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: FunmiOsho <60607637+FunmiOsho@users.noreply.github.com> Co-authored-by: lizaadebowale <liza.adebowale@nice.org.uk> Co-authored-by: L-Vazquez <37586672+L-Vazquez@users.noreply.github.com> * NXT-339 Update readme (#240) * NXT-353,NXT-343 404 errors on nextweb (#234) * NXT-343 Exclude next-js manifest files from url lowercasing middleware and unit test. Co-authored-by: John Holland * NXT-343 Modify comment to trigger TC build * NXT-343 Change url.pathname check to include _next/data and _next/static paths to avoid request errors * NXT-343 Add test for _next/data url condition * NXT-343 lowercase any mixed case indicators or guidance urls * NXT-343 Remove middleware page and unit tests * NXT-343 Hardcode indicators legacy redirects * NXT-343 Move indicators redirects from file directly to the config * NXT-343 Trying separate redirects file again * Revert "NXT-343 Trying separate redirects file again" This reverts commit f8d73685610fe09b773a9dd76bf3b26be8b8df01. * NXT-343 Correct redirects where there are duplicates with same case or missing lowercase version * Update package lock --------- Co-authored-by: John Holland <john.holland@nice.org.uk> Co-authored-by: John Davey <john.davey@nice.org.uk> Co-authored-by: John Davey <66066685+johndavey72@users.noreply.github.com> Co-authored-by: Colin M <12713935+colinmazu@users.noreply.github.com> Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: FunmiOsho <60607637+FunmiOsho@users.noreply.github.com> Co-authored-by: lizaadebowale <liza.adebowale@nice.org.uk> Co-authored-by: L-Vazquez <37586672+L-Vazquez@users.noreply.github.com> * NXT-345 Storyblok api key update (#239) * NXT-345: update initStoryblok to use new publicRunTime variable for SB. - add new yaml config variable - refactor catch all slug to logic in other pages Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * NXT-345: add test coverage for initStoryblok Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * NXT-345: add generic error as global const - update generic error message to global const - update tests to cover error handling Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * NXT-345: remove unused import * NXT-345 Linting fixes * NXT-345 update generic error message in SB util tests * NXT-345: remove server config properties and mocks --------- Co-authored-by: John Davey <johndavey72@users.noreply.github.com> Co-authored-by: John Davey <john.davey@nice.org.uk> --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: John Holland <john.holland@nice.org.uk> Co-authored-by: Claire Hunter <claire.hunter@nice.org.uk> Co-authored-by: John Davey <66066685+johndavey72@users.noreply.github.com> Co-authored-by: Dale Hudson <dalefrontend@gmail.com> Co-authored-by: John Davey <john.davey@nice.org.uk> Co-authored-by: John Davey <johndavey72@users.noreply.github.com> Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> Co-authored-by: Dale H <dale.hudson@nice.org.uk> Co-authored-by: Sue Garner <33061045+suegarner@users.noreply.github.com> Co-authored-by: Colin M <12713935+colinmazu@users.noreply.github.com> Co-authored-by: FunmiOsho <60607637+FunmiOsho@users.noreply.github.com> Co-authored-by: lizaadebowale <liza.adebowale@nice.org.uk> Co-authored-by: L-Vazquez <37586672+L-Vazquez@users.noreply.github.com>
lizaadebowale
added a commit
that referenced
this pull request
Jun 17, 2024
* Nxt 180 scaffold pages (#131) * NXT-180 Add basic routing; refactor fonts * NXT-180 Add basic Storyblok integration * NXT-180 Display SB data on home and about pages * NXT-180 Fix TS issues, abstract Storyblok connection --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nxt 181 card components (#135) * NXT-181 Card grid homepage WIP * NXT-181 Fix TS errors * NXT-181 Add card component from NDS * NXT-181 TS integration for cards and homepage * NXT-181 Resolve Storyblok link objects * NXT-181 Fix links when SB returns empty link objects * NXT-181 Alter card grid layout, fix failing tests * NXT-181 Remove logging --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nxt 185 hero and catnav (#136) * NXT-185 Add hero and catnav components * NXT-185 Add SSL proxy for Storyblok preview mode * NXT-185 Resolve draft versions from context * NXT-185 Refactor page fetching, add catch-all about route * NXT-185 Breadcrumbs WIP --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-185 Add unit tests, fix broken page fetching * NXT-187 Refactor breadcrumb code (#137) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-188 corporate page metadata (#139) * NXT-188 Fix failing tests * NXT-188 Add metadata from Storyblok, update user-generated metadata * NXT-188 Add canonical and language link/meta tags --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Bump commit * Bump commit * Bump * Bump commit * Fix broken snapshot * Replace public config with server config for SB tokens * NXT-186 Static paths WIP (#153) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nxt 186 storyblok static paths (#155) * NXT-172 Add API endpoint for deleting fs cache entries (#122) * NXT-155 Change alert functionality to read from InDev * NXT-155 Updated snapshot test * NXT-155 Added alert logic to docs/consultation tabs for project page * NXT-155 Add API endpoint for deleting fs cache entries * NXT-155 Prevent cacheing of API cache deletion response * NXT-155 possible caching fixes (#134) * NXT-155 possible caching fixes * NXT-155 stop it generating multiple cache entries in different cases * NXT-155 move lowercase fixes to getCacheKey function * NXT-155 Lint cache api; send group keys in 400 err * NXT-155 fix tests --------- Co-authored-by: John Holland <john.holland@nice.org.uk> Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: Claire Hunter <claire.hunter@nice.org.uk> * NXT-178 Add international enquiry form and test (#132) * NXT-190 embedded doesnt exist for hub pages (#138) * NXT-186 Static paths WIP * NXT-186 Update linting rules --------- Co-authored-by: John Holland <john.holland@nice.org.uk> Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: Claire Hunter <claire.hunter@nice.org.uk> Co-authored-by: John Davey <66066685+johndavey72@users.noreply.github.com> * Nxt 186 storyblok static paths (#156) * NXT-172 Add API endpoint for deleting fs cache entries (#122) * NXT-155 Change alert functionality to read from InDev * NXT-155 Updated snapshot test * NXT-155 Added alert logic to docs/consultation tabs for project page * NXT-155 Add API endpoint for deleting fs cache entries * NXT-155 Prevent cacheing of API cache deletion response * NXT-155 possible caching fixes (#134) * NXT-155 possible caching fixes * NXT-155 stop it generating multiple cache entries in different cases * NXT-155 move lowercase fixes to getCacheKey function * NXT-155 Lint cache api; send group keys in 400 err * NXT-155 fix tests --------- Co-authored-by: John Holland <john.holland@nice.org.uk> Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: Claire Hunter <claire.hunter@nice.org.uk> * NXT-178 Add international enquiry form and test (#132) * NXT-190 embedded doesnt exist for hub pages (#138) * NXT-186 Static paths WIP * NXT-186 Update linting rules * NXT-186 Resolve custom rendering method --------- Co-authored-by: John Holland <john.holland@nice.org.uk> Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: Claire Hunter <claire.hunter@nice.org.uk> Co-authored-by: John Davey <66066685+johndavey72@users.noreply.github.com> * Add verbose logging to debug test env * Auto clear SB cache * NXT-211 Add simple server-side catch-all route (#162) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-197 Add breadcrumbs (#164) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Temporarily circumvent SB cache * Temp page header component update * Update Storyblok types; ignore generated file with ESLint * Correct linting setup * Make grid gutter loose * Emit logs from status page check * Update status page logs * Replace test status page errors with warnings * Fix lockfile * Update tsconfig settings * Add new SB types * WIP info page & nested rich text component * Nxt 229 news article page (#174) * chore: test updating the storyblok types from npm script * feat: test adding image to the storyblok hero and test using button as link if ctaLink and ctatext returned from Storyblok * chore: add storyblok richtext renderer as dep * feat: add NewsArticle component to render news articles in app - Add News Article component with PageHeader and richtext renderer for content. - Update PageHeader to render formatted date from design Dale H * style: test adding basic layout styles from scss * style: update image css * feat: first pass at blockquote we will use in the richtext examples in news articles * feat: test adding blockquote to the richtext renderer resolver * feat: first pass at NewsLetterSignup using DS components * styles: temp update to blockquote component style * feat: first pass at adding route for news with catch all for news-blogs-and-podcasts (TEMP till slug sturcture agreed) * chore: remove console log * refactor: update NewsArticle component to start using DS components * feat: first pass adding BlogPost component to render blog pages (TODO: move to its own route under pages dir) * refactor: remove empty h2 tag * feat: flesh out news article and move to correct path - Updated the news article to show panels on sidebar - updated StoryblokPageHeader to include tags and date (temporary test until new PageHeader version completed) - Added first pass on StoryblokImage component to use Storyblok image service, adds support for webp and jpeg formats Dale H * feat: first pass on sidebar panels for related news and links - Added first pass StoryblokRelatedLink and StoryblokRelatedNewsLink components - Added Panels to sidebar on NewsArticle Page TODO: update spacing across news article and it's components TODO: add tests for components and pages Dale H * refactor: update related news and link components to use Link component from next-web * chore: remove console.logs * feat: first pass at updating breadcrumbs - Update logic on breadcrumb function - Add path on getBreadcrumbs call, using partly hard coded path. TODO: get path dynamically, within getStaticProps Dale H * refactor: test PageHeader alpha release in StoryblokPageHeader * styles: update featuredImage offset styles to use custom property, so we can update the variable value inside of media queries * feat(ts): udpate type for image and pass rest prop to handle passing of common image attributes * refactor: update newsarticle image to take width and height and maintain 16/9 aspect ratio * test: update image component test and add fallback image for when src is empty * refactor: hardcode Breadcrumbs in news article. Comment out breadcrumb logic from StoryblokPageHeader to test news article * fix: remove updatedBreadcrumb * feat: start richtextRenderer component * styles: remove max width from blockquote to span full width of container * fix: revert StoryblokPageHeader component for stories that use the StoryblokPageHeader blok * feat: first pass on NewsPageHeader to use on news, in-depth, podcast and blog pages * feat: update richtext and news article components - Updated types for richtext component - Add NewsPageHeader and updated Richtext component to NewsArticle page component Dale H * styles: update style to handle images contained within richtext content * feat: first pass at YoutubeEmbed handled by rich text renderer * refactor: update newspageheader to use date util from repo * feat: Update richtext component to render inline images as StoryblokImage component ` * fix: add workaround to handle inline quotes in the richtext field of news stories NOTE: currently the children of the inline quote doesn't render correctly when using the blockquote component in the nodeResolver of storyblok-rich-text-renderer. We bump into a few issues around rendering and correct typing. I've hardcoded the html and imported styles so we can render the correctly styled quote TODO: Update the blockquote component or richtext types to handle passing through richtext quote children to the blockquote component * fix: update richtext interface, revert to initial interface from previous commit * tests: add tests for breadcrumbs and pageheader meta for tag and published date * fix: update breadcrumb name and resources link panel * test: remove test for fetchPriority attribute * refactor: update the inner Link component to use href over to attribute * tests: add relatednewslink tests * tests: first pass adding youtube embed tests TODO: add a field in storyblok for title so we can add an accessible title to the video embed Dale H * tests: firstpass adding tests for StoryblokRichText component * tests: first pass adding tests for Storyblok NewsArticle component * styles: update cta button so inner text wraps * fix: update types for newsarticle, add test to check whether signup action is rendered * tests: update NewsArticle tests * tests: first pass test news article slug page * fix: remove unused imports from richtext component and tests * chore: update design system version for new pageheader * refactor: update the newsarticle component to take new page header over newspageheader. - Update NewsArticle.tsx to use design system PageHeader new version - Add hardcoded breadcrumb values and handle in NewsArticle to be passed to PageHeader component TODO: Possible refactor of NewsPageHeader to be a generic component that can be used on news index and slug pages TODO: Image spacing to be handled via useEffect and ref to image to dynamically update the css varaible for its spacing Dale H ` * chore: rename pageheaderfooter that contains date and tag * refactor: update StoryblokImage and NewsArticle for overlap Added refs to NewsArticle and StoryblokImage so we can target them for updating the custom property value for the image overlap between header and body * style: update spacing around pageheader elements * tests: add further coverage on NewsArticle component * fix: udpate relatedlink and relatednewslink to use resolveStoyblokLink util function * round the pixels down on the featuredImageOffset value * fix: append /m/ for webp source image * fix: add title to Youtube embed for screen reader use * chore: update storyblok types * fix: update youtube iframe embed source value, remove comments * NICE-170 latest from nice & homepage (#175) * NICE-170 Homepage WIP * NICE-170 Latest news WIP * NICE-170 List latest news articles * NICE-170 Featured story WIP * NICE-170 Implement homepage hero * NICE-170 Action banner WIP * NICE-170 Resolved link, promo box WIP * NICE-170 Fix broken links for story objects * NICE-170 Complete "latest from NICE" section * NICE-170 Revert tsconfig, add NewsStory types * NICE-170 Add unit tests for homepage news section & components * NICE-170 Render news grid image as link, fix snapshots * NICE-170 Use better semantics, fix snapshots * NICE-170 misc fixes post review --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nxt 229 updates and fixes (#177) * fix: add empty dependancy array to useEffect to run once on mount, resize event for image * fix: add missing slash on jpeg src * feat: first pass adding author blok * styles: round image for author image * refactor: bring in update to fetchStory function and add author blok * first pass onblog post with listed authors - rough draft * update author component, extract the authorlist from blog post * update interface for blogpost and remove the inlined authorlist * fix: remove trailing comma from tsconfig * refactor: move authorlist to its own component, test author component * add authorList component and extend tests * start moving newspage header and child component into own files * fix: rename blogpost and newsarticle component to keep naming convention for Storyblok blok components * refactor: update AuthorList and tests * tests: start blogpost component tests * styles: update richtext elements vertical spacing and blog post * tests: update blog post slug page tests * tests: update youtube embed tests to include updated title property * tests: add missing slash from returned image source in tests * tests: update snapshot tests related to pageheader updates * styles: remove need for nested grid on BlogPost component, spacing updates on articles and posts * tests: update snapshots related to pageheader updates * tests: add testid to pageheader to test id applied, after pageheader updates * style: add commented styles to take over pageheader secondsection styles in NDS * feat: first pass at StoryblokIframe component * spike: spike nextjs/third-parties and react-lite youtube embeds - Tested next third parties, works but difficulty testing - Tested react-lite-youtube-embed, works and tested. TODO: test accessibility of react-lite-yt-embed further and against standard iframe implementation. TODO: ensure component is typed correctly * chore: remove spiked nextjs/third-parties * fix: remove incomplete import from richtext styles causing no styles to load * feat: use react-lite-youtube-embed for YT embed component * fix: update params and cookie props on on LiteYoutubeEmbed Issue with Safari not autoplaying the video once the poster play button was clicked. Adding cookie=true and updating params to include autoplay=1 solves this issue. * styles: update action banner spacing on newsarticle * styles: first pass normalise spacing to use DS spacing values * styles: normalise blockquote spacing * styles: tidy duplicated spacing * fix: add lodash debounce to handleResize functions on blog and news page components * tests: update breadcrumb test on slug page, and remove expect from youtubeembed test * NXT-250 Refactor fetchstories utility function (#179) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-251 Refactor NewsCard component (#180) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nxt 229 update tests and mocks (#181) * update test utils mocks for blog post and news article * fix: update type of blogpost blok and update tests to use mock util data * fix: udpate type of slug pages for blog and news to pass correct blok types * fix: udpate types and mock for newsarticle component - Update blok prop type and revert back to types generated by Storyblok - remove previous override type for news article - Update StoryblokNewsArticle to use mock data from test utils Dale H * refactor: update authorlist to render author blok over stoyblok component, and add className prop to AuthorBlok * tests: update test to check for author data in blogpost component * tests: update slug page tests on news and blog page slugs, using mock data from utils * tests: add richtext mock data and update tests that use richtext * NICE-169 secondary promo (#182) * NICE-169 Promo sections WIP * NICE-169 Complete Promo Box & tests --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nice 222 news blog style fixes (#184) * styles: update horizontal spacing * styles: update targetting on Youtube embed when child of richtext component * styles: update spacing on featuredImage and page header * refactor: normalise grid layout between newArticle and blogPost, update featuredImage margins * styles: switch featuredImage to 21/9 between sm and md breakpoints * styles: udpate spacing between main content/sidebar, header and action banner * styles: update bottom margin on youtube embed in richtext * styles: add fallback styles for inset and set button hover cursor to pointer * styles: update author styles from crit * styles: remove pageheader bottom margin * NICE-164 Implement Storyblok action banner (#185) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Big snapshot update * NICE-226 draft info page (#190) * NICE-226 Draft info page template * NICE-226 Revert Space ID * NICE-226 Remove log --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Fix linting error * Fix stylelint errors * Style fixes (#191) * fix: update styles to remove BEM convention in modules where we can * fix: add ignore on selector-class-pattern for targeting page header classes from DS in News and Blog components * Stylelint config update * NICE-178 Build the news listing template (#183) * NICE-178 Retrieve list of news articles from storyblok - wip * NICE-178 Add pagination component - wip * NICE-178 Return object with stories and total pages from storyblok response * NICE-178 Add current page and pagination destinations * NICE-178 Calculate total pages for pagination * NICE-178 Redirect to news/articles/ if page is outside bounds * NICE-178 Destructure gss props, return props directly * NICE-178 Componentise news list pagination * NICE-178 Rename prop as they aren't optional * NICE-178 Unit test newsListPagination wrapper * NICE-178 Revert tsconfig to fix issue with module resolution * NICE-178 Unit testing wip. Add npm script so space ID can be set and exported from terminal * NICE-178 Unit tests * NICE-178 Add action banner * NICE-178 Checkout web package.json etc to avoid merge conflict * NICE-178 Add NewsGrid component to articles list layout * NICE-178 Repurpose NewsGrid as NewsList for list pages * NICE-178 Move return in fetchStories to match storyblok-main * NICE-178 Add isListItem variant for news list. Setup basic style and layout for newslist page * NICE-178 Add featured story, sort stories by content.date descending. Update type useage * NICE-178 Add newslist nav component * NICE-178 Add npm script for updating types via shell * NICE-178 Update types * NICE-178 Add SEO * NICE-178 Adjust mobile styles for newscard * NICE-178 Add unit tests. Update snapshots * NICE-178 Add news nav to news index page. Change getActiveLink to match url paths exactly * NICE-178 Fix unit tests * NICE-178 Remove logging line * NICE-178 Move components to News dir to declutter main dir * NICE-178 Update location of pagination component * NICE-178 Prettify * NICE-178 Add unit tests for isNewsListItem variant * NICE-178 Comment out noisy router test * NICE-178 Skip first story on first page as it's featured. Articles index unit test - wip * NICE-178 Remove logger lines * NICE-178 Add todo for gssp on articles index * NICE-178 Unit tests * NICE-178 unit test gssp * NICE-178 remove worthless unit test * NICE-178 Update unit tests. Adjust border thickness for NewsCard * NICE-178 fix nesting level of selectors picked up by linting rule * NICE-178 Move storyblok utils mocks to outer describe * NICE-178 Remove magic number from unit test * NICE-178 Remove unnecessary string conversion in query * NICE-178 Handle storyblok fetchStories error condition * NICE-178 Revised error message * NICE-231 homepage update (#193) * NICE-231 Add homepage spacing, update promo box transparency * NICE-231 Complete Spotlight component * NICE-231 Update spotlight, add new news types * NICE-231 Fix rendering of homepage news links * NICE-231 Update snapshots * NICE-231 Update spotlight component to handle podcasts * NICE-231 Update snapshots, temporarily remove articles listing page unit tests * NICE-231 Fix TS errors * NICE-231 Homepage style updates * NICE-231 Fix snapshots --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Fix linting errors * Fix Card Grid link resolution * Nice 215 blog listing build (#195) * test: udpate snapshot test on newsarticle after updating mock data * udpate news and blog pages to filter scheduled stories according to date field * start validate route/pagination work * NICE-215 validateRouteParams wip * update types around validateRouteParams for storyblok * NICE-215 Handle no stories, parameterise validateRouteParams items and defaults * NICE-215 Fix starts_with after debug * test redirect and update blog page - first pass testing path redirect for out of range pagination pages - update blog listing page with new validateRouteParams - update news article listing page with resolvedUrl Authors: John Davey Dale Hudson * NICE-215 Remove debugging, tidy up out of range function * NICE-215 unit test featured story prop in gssp * NICE-215 pass sbParams differently, clean up types * NICE-215 Handle author content in FeaturedStory component * NICE-215 Change friendlyDate from toLocaleDateString to Intl.DateTimeFormat as this is more performant * NICE-215: update styles and destructure props for author in featuredStory * NICE-215: add first pass at author in newscard * NICE-215: update where we set AuthorHeadingElement * NICE-215: fix no rendering of secondSection if no authors length * NICE-215(fix): layout of author block * update storyblokimage for better handling of serviceOptions * NICE-215: update author card with iscardauthor and headinglevel props - update author to handle different heading levels with default to h4 - add prop to handle author when in card, updated styles for this too * NICE-215: update featured card with updated author component * remove redundant function and logs from SB utils * NICE-215(fix): update how author is handled in blogpost now author component updated * NICE-215(styles): udpate author styles in news and featuredStory cards * NICE-215: update full_slug link urls to absolute urls * NICE-215: udpate perf of background image and author keys - Added Storyblok image service parameters to resize and reduce quality of images to improve image loading times - Changed id for author lists within news and featured story cards * NICE-215: revert jpeg sources ready to refactor to use all image service options * NICE-215: add iframe component to storyblok init * NICE-215(fix): update image to ternary for correct loading of fallback image * NICE-215: fix Youtube embed button aria - remove announce prop from youtube embed, was duplicating aria label values - add fireevent to button click event in YT embed test * NICE-215(refactor): Update StoryblokImage component - Update image component to handle Storyblok image service options - As first pass we only cater for service options width, height, smart, quality - Update components using the StoryblokImage component with updated serviceOptions so we can optimise images currently in use. Dale H * NICE-215: handle error for empty stories array --------- Co-authored-by: John Davey <john.davey@nice.org.uk> Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * NICE-238 Add podcast page (#194) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * return null if author is string (#196) * Nice 215 fix tests (#197) * NICE-215: fix background images using SB image service by encoding brackets () * NICE-215: remove author as string from newsArticle as not part of content model * NICE-215(fix): Update snapshots for link and bgimg - updated snapshots for updated absolute url link handling in news and featured cards - updated snapshots for encoded background image urls containing storyblok image service filters - updated snapshots for featured and news cards with added authors John Davey Dale Hudson * fix pagination tests with updated prop names * NICE-215(fix): update snapshots from image component updates * NICE-215(fix): udpate tests after AuthorList refactor * NICE-215(fix): handle blogpost authors when array empty Handle blog posts author when they are pubilshed with an unpublished or no author attached. John D Dale H * Nice 231 homepage update (#198) * NICE-231 Add homepage spacing, update promo box transparency * NICE-231 Complete Spotlight component * NICE-231 Update spotlight, add new news types * NICE-231 Fix rendering of homepage news links * NICE-231 Update snapshots * NICE-231 Update spotlight component to handle podcasts * NICE-231 Update snapshots, temporarily remove articles listing page unit tests * NICE-231 Fix TS errors * NICE-231 Homepage style updates * NICE-231 Fix snapshots * NICE-231 Resolve merge conflicts * NICE-231 Homepage UI updates * NCIE-231 Limit news card to 1 author * NICE-231 Update author UI, change absolute URLs * NICE-231 Fix snapshots * NICE-231 Update Promobox mock data and snapshot --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nice 217 podcast listing page (#199) * NICE-217: podcast listing page, initial setup - Condtionally render newscard image - Update newscard author, return null if author is string D Hudson J Davey * NICE-217 Update podcast listing page - Make newsCard image optional - Fix broken unit test by removing redundant console log - Early return if author is just a string - Make newsListNav propless with destinations contained as they won't change - First pass at podcast listing page Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NICE-217 Add unit tests to cover optional showImage Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NICE-217 Update lead text on podcast list page * NICE-217: update breadcrumbs with absolute links --------- Co-authored-by: John Davey <john.davey@nice.org.uk> Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NICE-248 Resolve author linting issues (#200) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NICE-218 First pass at in-depth listing page (#201) Co-authored-by: John Davey <john.davey@nice.org.uk> Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * Nxt 270 news homepage (#202) * NXT-270 News homepage WIP * NXT-270 More news homepage WIP * NXT-270 Complete news homepage --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Fix linting issue * Add remote pattern image config * NICE 253 Refactor init storyblok (#204) * proof that separating initStoryblok enables proper mocking * streamline component imports for storyblokcomponent pages Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> --------- Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NICE-253 Fix issue with initStoryblok refactor breaking the catch all slug. Optimise component object for lean bundle. (#207) Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NXT-269(fix): filter articles out of latest news with future date (#208) * NICE-254 Handle image optimisation (#205) * NICE-254 Optimise images coming from storyblok image service. Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NICE-254 Update snapshots * NICE-254 Add optimise image util and unit test * NICE-254 Reinstate storyblokhero --------- Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * Nxt 292 blog page fixes from feedback (#209) * NXT-292: fix blog post author and author list headings * NXT-292: fix headings, not to render when no content * Merge storyblok-main Co-authored-by: John Davey <johndavey72@users.noreply.github.com> --------- Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * Nxt 292 blog page fixes from feedback (#210) * NXT-292: fix blog post author and author list headings * NXT-292: fix headings, not to render when no content * Merge storyblok-main Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * NXT-292 linting * NXT-292 merge sbmain --------- Co-authored-by: John Davey <johndavey72@users.noreply.github.com> Co-authored-by: John Davey <john.davey@nice.org.uk> * NXT-292: update homepage snapshot * NICE-255 Change enablejsapi from 0 to 1 (#212) * NICE-250 Managing focus on pagination route changes (#214) * spike pagination announcement for screenreader users - Add a div calling out the current page and total pages when navigating via screenreader - Update focus to hidden div, just above NewsList - this is one option of a few we need to test, preferably with screenreader users. - Options to go through linked on ticket: NICE-250 * NICE-250 Use Announcer component for pagination changes * NICE-250 add aria-hidden on focus element to prevent screen reader reading out 'blank' * NICE-250 Add announcer to other list pages. Pagination focus hook wip Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NICE-250 Roll out pagination focus to other list pages. Add component for pagination focus. * NICE-250 Remove console log * NICE-250 Add unit tests for pagination focus component * NICE-250 Refactor announcer code into a re-useable component * NICE-250 Remove unused deps and commented out code --------- Co-authored-by: dalecreativ <dalefrontend@gmail.com> Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NICE-260 Calculate canonical urls for slug pages and unit test (#216) Co-authored by Dale Hudson * Nxt 283 font size fixes (#215) * NXT-283: Update root font size to handle narrow viewport sizes. Co-authored by: John Davey * NXT-283: update font sizes for h1 on smaller screen sizes Targetting hero and page headings for home, news article, and blog post pages. Co-authored by: John Davey * NXT-283 Adjust h1 font-size for narrow viewport. Apply word-break and hyphenation for very narrow viewports Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> --------- Co-authored-by: John Davey <john.davey@nice.org.uk> Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * Nice 261 refactor image optimisation (#217) * NXT-261: refactor constructStoryBlokImageSrc to encodeParens and handle background images Co-authored by: John Davey. * NXT-261: null check image on podcast slig page * NXT-261: remove optimiseImage import from tests * NICE-261: update failing snapshot from encodeParens update (#218) * Nice 266 homepage spacing (#221) * NICE-266 Update homepage spacing; restore blog authors on news homepage * NICE-266 Miscellaneous QA fixes * NICE-266 Update blog page title * NICE-266 Set podcast image to unoptimised * NICE-266 Prevent future articles from appearing on news homepage * NICE-266 Pedantry --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nice 266 homepage spacing (#223) * NICE-266 Update homepage spacing; restore blog authors on news homepage * NICE-266 Miscellaneous QA fixes * NICE-266 Update blog page title * NICE-266 Set podcast image to unoptimised * NICE-266 Prevent future articles from appearing on news homepage * NICE-266 Pedantry * NICE-266 Set base URL for podcast image --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nice 266 homepage spacing (#224) * NICE-266 Update homepage spacing; restore blog authors on news homepage * NICE-266 Miscellaneous QA fixes * NICE-266 Update blog page title * NICE-266 Set podcast image to unoptimised * NICE-266 Prevent future articles from appearing on news homepage * NICE-266 Pedantry * NICE-266 Set base URL for podcast image * NICE-266 Pluralise podcasts label --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NICE-283 spacing again (#229) * NICE-283 Make spacing and content changes * NICE-283 Spacing, routing, text updates * NICE-283 A11y fixes * NICE-238 Amend news article action banner spacing * NICE-283 Update action banner spacing * NICE-283 Restore access token config --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Update snapshots * NXT-276 Update default image with base URL from config * NXT-217 Use Ocelot for Storyblok cacheing (#225) * NXT-217 Installed Ocelot.Administration NuGet package * NXT-217 Added Storyblok config to Ocelot routes * NXT-217 Configure Storyblok to use Ocelot * NXT-217 Add Ocelot Administration endpoint * NXT-217 Refactor Ocelot cache config settings * NXT-217 Add Ocelot for Storyblok section to readme * NXT-217 Update Readme --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NICE-291 Return 404 for root catch all unless env var enables it (#230) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-346 Revert env var config, add test page (#232) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nxt 346 revert env vars (#233) * NXT-346 Revert env var config, add test page * NXT-346 Add test env var to config --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * fix: update default ocelotEndpoint to empty value * Remove env var from next config * NXT-319, NICE-251 unit testing utility functions and news index pages, adding response status from SB API (#231) * initial commit * add boilerplate for get server side props * update validaterouteparams to return redirect if page is less than 1 TODO: alternative approach of assigning page to 1 if query.page is less than 1 * update mock data and refactor mocks for listing page * stub out getStoryblokAPi get method and modify response per test * Added json files for various responses Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * Added further test for fetchStory util co-authored by Dale Hudson * NXT-319 Add server error unit test Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NXT-319 Add unit tests for fetchStories co-authored by Dale Hudson * NXT-319 Add fetchLinks unit tests Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NXT-319 Add fetchLinks unit tests Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NXT-319 Remove promise.reject from fetchStories to prevent unreachable code issue. Tidy up imports in storyblok utils. Co-authored by Dale Hudson * NXT-319 Change mock links success response to match item returned from storyblok/react. Add note to investigate jest setup mocking issue * NXT-319 Add getBreadcrumbs unit tests * NXT-319 Add error handling to friendlyDate. Unit test friendlyDate * NXT-301: add test coverage for getNewsTypes * NXT-319: update validateRouteParams query.page to be set to one if less than 1 * NXT-319: start validateRouteParams test coverage * NXT-319: update news index page tests after updates * NXT-319: test for success and error responses from fetchStories returned to valiudateRouteParams * NXt-319: remove skip test validateRouteParam * NXT-319 Handle no stories on index pages. Return not founds for incorrect query page param. Refactor and add unit tests for validateRouteParams. Remove log lines. Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NXT-319 Remove unused spy. Add unit test todos for news articles listing" * NXT-319 Add unit test to check undefined return from getSlugFromParams. Remove unused dep * NXT-319 Add unit tests for news article list page. Refactor around thrown errors from validateRouteParams. Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NXT-319 Unit tests of news articles renderins. Gssp correct props unit test. * NXT-319 Rendering unit tests for news articles listing Co-authored by: Dale Hudson * NXT-319 Convert in-depth gssp to try catch format. Adjust types to separate error props from success props. * NXT-319 Move isError util function to storyblok utils and unit test. Make public facing error messages more generic". Update snapshot. * NXT-319 Update blog posts index gssp to try catch format. Separate error and success props and types" * NXT-319: update podcast gssp to handle errors from api * NXT-319 Remove resolved url from gssp args. Update unit tests to reflect removal of API details in error messages. * NXT-319 Add unit tests for news index pages * NXT-319 Add further unit tests for news index pages. * NXT-319 Refactor blog slug page. Added blog slug page unit tests. Added mock success response for blog post. * NXT-319 Refine types for fetchStory, blog slug page and unit test mock data types. Remove error property from fetchStories. Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NXT-319 Remove unused deps * NXT-319: remove test backup files * NXT-319: refactor news article slug with TODO comments for discussion * NXT-319: add test coverage for articles slug page * NXT-319: add TODO for discussion on fetchStory * NXT-319: add snapshot for news article page * NXT-319: remove redundant imports from news slug tests * NXT-319: add mockData for podcast page * NXT-319: refactor podcast slug page in line with other slug pages * NXT-319: add tests for podcast slug page with snapshot * NXT-319: udpate type for newsarticle fetch and props. Add logger on getadditionalMetaTag failure * NXT-319: add useMemo back to getAdditionalMetatags in blog post page * NXT-319: add logger to getAdditionalMetaTags in podcast page * NXT-319: remove unused imports from news slug page * Add further test coverage for podcast slug page - updated snapshots Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * NXT-319: remove redundant mocks from blog slug page tests * NXT-319 Error handling for news index and unit test - wip * NXT-319: test news homepage, gssp and error handling * NXT-319: add tests for homepage and handle fetch errors from API. Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * NXT-319: fix linting errors Co-authored by: John Davey * NXT-319: update error handling in utils. consistant messaging and var naming * NXT-319: remove TODOs * NXT-319: update homepage snapshot and test with error message update Co-authored by: John Davey * NXT-319: replace isError from utils with lodash isError equivalent Co-authored by: John Davey * NXT-319: remove isError import from test * NXT-319: move TODOs to tech debt and remove redundant comments. - update podcast listing mock data Co-authored by: John Davey --------- Co-authored-by: John Davey <john.davey@nice.org.uk> Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * Nxt 301 blog page fixes from feedback (#219) * NXT-301: order list of authors alphabetically * NXT-301: add border and spacing to author section in blog header * NXT-301: update author section spacing and remove bottom margin from any richtext component last child element * NXT-301: remove TODOs that have been resolved through updates or discussions * NXT-301: increase on author name if img is included * NXT-301: add role presentation to images carrying no alt text * NXT-301: update width of PageHeader fullwidth-container Full width container updates changed the alignment of pageheader to the rest of content on blog and news articles. Updated so content between header and main content is aligned. * NXT-301: add stylelint ignore rule for global styles targetting page-header * NXT-301: update author blok border with token for colour * NXT-301: revert page header container horizontal spacing. * NXT-301: pageheader author block spacing - Only onclude border and bottom padding over breakpoint * NXT-301: first pass at updating the richtext image aspect ratios based on storyblok image width and heights * NXT-301: update test data to have accurate url structure for SB CDN images * NXT-301: update displayed dates to use the time tag. - update components that display dates to use time tage - update the mock data for dates - refactor the news/blog header meta to use getNewsType util TODO: refactor tag and date into meta component for reuse across other components Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * NXT-301: update richtext image - update aspect ratio, width and height attrs using SB image url path to get width and height * NXT-301: replace storyblok cdn and link domain from url mock data. Mitigate calls to actual url. * NXT-301: upate storyblok-data mock urls * NXT-301: update snapshots after mock data changes to urls * NXT-301: update styles to nudge featured image - Nudge pixels on page header and featuredimage across news and blog page templates - Keeps margin top within 2rem of page header bottom content TODO: revisit handling of featuredImage offsets. Adding to tech debt for post go live Dale H * NXT-301: remove TODOs and put in tech debt log for discussion * NXT-301: move setComponent causing rendering issues --------- Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * fix: add back lint disable line for global targetting of css from DS * Main merge (#242) * NXT-246 Indicator Old URL mixed casing - redirects not working correctly (#192) * NXT-246 Added middleware to convert all resolved urls to lowercase Next js is routing to different pages/components if the url uses mixed casing compared to lowercase. This fixes that. * NXT-246 Fixed check to compare mapped indicator url with resolved url * NXT-246 Fixed linting issues * NXT-246 Added unit test for middleware function * NXT-330 Indicators change chapters to chapter (#226) * change chapters to chapter in indicators [slug] and update unit tests * NXT-330 Removed obsolete snapshots --------- Co-authored-by: John Davey <john.davey@nice.org.uk> * NXT-337 Fix for empty product panel (#227) * NXT-337 Fix for missing _embedded.niceIndevPanel Undefined when no documents are present. * NXT-337 Further null checks for panel list * NXT-343 Exclude next-js manifest files from url lowercasing middlewar… (#228) * NXT-343 Exclude next-js manifest files from url lowercasing middleware and unit test. Co-authored-by: John Holland * NXT-343 Modify comment to trigger TC build * NXT-353 Remove middleware page (#235) * NXT-353 Remove middleware page * NXT-353 Reinstate error causing middleware files * NXT-242 Add Terraform config (#176) * NXT-210 Add test logs to status page * NXT-210 Move logging test into forms dir * NXT-242 Add Terraform config * NXT-242 Update vars * NXt-242 Change ip to non public * NXT-241-new k6 scripts - js files and json files * NXT-242 Change bucket name * NXT-242 Add values into json * NXT-242 Update tfvars * NXT-242 Change to octo vars * NXT-242 Update tf vars * NXT-242 Fix typo * NXT-242 Fix issues * NXT-242 Update tfvars * NXT-241-updates to perfomance test k6 scripts * NXT-241-revert changes to vscode settings * NXT-241-remove typeahead terms from combined scripts and cleanup * NXT-241-added more paths to the combined json file * NXT-241-edited galp_pages test data * NXT-242 Added load balancer config to tf * NXT-242 Add build number to config dir * NXT-242 Added diagnostic info * NXT-242 Update efs volume path * NXT-242 Update var names * NXT-242 Add node env * NXT-242 Refactor TF * NXT-242 Revert config.yml * NXT-242 Update default.yml * NXT-242 Remove comment * NXT-242 Variablise CPU and Memory * NXT-242 Remove breaking characters * NXT-242 Add variable characters * NXT-242 Change variable type to number * NXT-242 Move pino-mqrc.json * NXT-242 Fix pino-mq config problem * NXT-242 Fix issues with pino-mq * NXT-288 Enable AWS Exec # Conflicts: # aws/terraform/terraform.tfvars # aws/terraform/variables.tf * NXT-242 Add server count variable, remove cloudwatch variable * NXT-241 Add basic test and change dir * NXT-241-fix-merge-commits * NXT-241-add-more-test-data * NXT-241-add-stages-plus-the-noConnectionReuse * NXT-241-prettier-fixes * NXT-242 Add teamcity aws config * NXT-242 Add extra stuff to k6 testing file * NXT-347 Update testing files * NXT-242 Add gitignore * NXT-242 Add readme.md * NXT-350 Add curl to Docker * NXT-350 Add healthcheck * NXT-242 Remove PM2 * NXT-242 Update readme * NXT-210 Update logtest.page.tsx * NXT-242 Remove performance testing * NXT-242 Apply TF format * NXT-242 Fix typo * NXT-242 Add comments to readme * NXT-242 Update readme --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: FunmiOsho <60607637+FunmiOsho@users.noreply.github.com> Co-authored-by: lizaadebowale <liza.adebowale@nice.org.uk> Co-authored-by: L-Vazquez <37586672+L-Vazquez@users.noreply.github.com> * Update package lock --------- Co-authored-by: John Holland <john.holland@nice.org.uk> Co-authored-by: John Davey <john.davey@nice.org.uk> Co-authored-by: John Davey <66066685+johndavey72@users.noreply.github.com> Co-authored-by: Colin M <12713935+colinmazu@users.noreply.github.com> Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: FunmiOsho <60607637+FunmiOsho@users.noreply.github.com> Co-authored-by: lizaadebowale <liza.adebowale@nice.org.uk> Co-authored-by: L-Vazquez <37586672+L-Vazquez@users.noreply.github.com> * Nice 249 sitemap generation (#245) * NICE-249 Fetch news content for sitemap from Links API, unignore sitemaps from Git, edit static sitemap from Orchard * NICE-249 Complete sitemap generation * NICE-249 Scrub auto-generated sitemap --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-345 Consolidate preview and public access tokens, remove env test page (#246) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-369 Add redirects for news section (#247) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Remove cv parameter from sbParams if Ocelot not present * Fix incorrect news redirect * Fix failing test due to CV parameter * Storyblok merge main (#250) * NXT-246 Indicator Old URL mixed casing - redirects not working correctly (#192) * NXT-246 Added middleware to convert all resolved urls to lowercase Next js is routing to different pages/components if the url uses mixed casing compared to lowercase. This fixes that. * NXT-246 Fixed check to compare mapped indicator url with resolved url * NXT-246 Fixed linting issues * NXT-246 Added unit test for middleware function * NXT-330 Indicators change chapters to chapter (#226) * change chapters to chapter in indicators [slug] and update unit tests * NXT-330 Removed obsolete snapshots --------- Co-authored-by: John Davey <john.davey@nice.org.uk> * NXT-337 Fix for empty product panel (#227) * NXT-337 Fix for missing _embedded.niceIndevPanel Undefined when no documents are present. * NXT-337 Further null checks for panel list * NXT-343 Exclude next-js manifest files from url lowercasing middlewar… (#228) * NXT-343 Exclude next-js manifest files from url lowercasing middleware and unit test. Co-authored-by: John Holland * NXT-343 Modify comment to trigger TC build * NXT-353 Remove middleware page (#235) * NXT-353 Remove middleware page * NXT-353 Reinstate error causing middleware files * NXT-242 Add Terraform config (#176) * NXT-210 Add test logs to status page * NXT-210 Move logging test into forms dir * NXT-242 Add Terraform config * NXT-242 Update vars * NXt-242 Change ip to non public * NXT-241-new k6 scripts - js files and json files * NXT-242 Change bucket name * NXT-242 Add values into json * NXT-242 Update tfvars * NXT-242 Change to octo vars * NXT-242 Update tf vars * NXT-242 Fix typo * NXT-242 Fix issues * NXT-242 Update tfvars * NXT-241-updates to perfomance test k6 scripts * NXT-241-revert changes to vscode settings * NXT-241-remove typeahead terms from combined scripts and cleanup * NXT-241-added more paths to the combined json file * NXT-241-edited galp_pages test data * NXT-242 Added load balancer config to tf * NXT-242 Add build number to config dir * NXT-242 Added diagnostic info * NXT-242 Update efs volume path * NXT-242 Update var names * NXT-242 Add node env * NXT-242 Refactor TF * NXT-242 Revert config.yml * NXT-242 Update default.yml * NXT-242 Remove comment * NXT-242 Variablise CPU and Memory * NXT-242 Remove breaking characters * NXT-242 Add variable characters * NXT-242 Change variable type to number * NXT-242 Move pino-mqrc.json * NXT-242 Fix pino-mq config problem * NXT-242 Fix issues with pino-mq * NXT-288 Enable AWS Exec # Conflicts: # aws/terraform/terraform.tfvars # aws/terraform/variables.tf * NXT-242 Add server count variable, remove cloudwatch variable * NXT-241 Add basic test and change dir * NXT-241-fix-merge-commits * NXT-241-add-more-test-data * NXT-241-add-stages-plus-the-noConnectionReuse * NXT-241-prettier-fixes * NXT-242 Add teamcity aws config * NXT-242 Add extra stuff to k6 testing file * NXT-347 Update testing files * NXT-242 Add gitignore * NXT-242 Add readme.md * NXT-350 Add curl to Docker * NXT-350 Add healthcheck * NXT-242 Remove PM2 * NXT-242 Update readme * NXT-210 Update logtest.page.tsx * NXT-242 Remove performance testing * NXT-242 Apply TF format * NXT-242 Fix typo * NXT-242 Add comments to readme * NXT-242 Update readme --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: FunmiOsho <60607637+FunmiOsho@users.noreply.github.com> Co-authored-by: lizaadebowale <liza.adebowale@nice.org.uk> Co-authored-by: L-Vazquez <37586672+L-Vazquez@users.noreply.github.com> * NXT-339 Update readme (#240) * NXT-353,NXT-343 404 errors on nextweb (#234) * NXT-343 Exclude next-js manifest files from url lowercasing middleware and unit test. Co-authored-by: John Holland * NXT-343 Modify comment to trigger TC build * NXT-343 Change url.pathname check to include _next/data and _next/static paths to avoid request errors * NXT-343 Add test for _next/data url condition * NXT-343 lowercase any mixed case indicators or guidance urls * NXT-343 Remove middleware page and unit tests * NXT-343 Hardcode indicators legacy redirects * NXT-343 Move indicators redirects from file directly to the config * NXT-343 Trying separate redirects file again * Revert "NXT-343 Trying separate redirects file again" This reverts commit f8d73685610fe09b773a9dd76bf3b26be8b8df01. * NXT-343 Correct redirects where there are duplicates with same case or missing lowercase version * Update package lock --------- Co-authored-by: John Holland <john.holland@nice.org.uk> Co-authored-by: John Davey <john.davey@nice.org.uk> Co-authored-by: John Davey <66066685+johndavey72@users.noreply.github.com> Co-authored-by: Colin M <12713935+colinmazu@users.noreply.github.com> Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: FunmiOsho <60607637+FunmiOsho@users.noreply.github.com> Co-authored-by: lizaadebowale <liza.adebowale@nice.org.uk> Co-authored-by: L-Vazquez <37586672+L-Vazquez@users.noreply.github.com> * Storyblok main merge conflicts (#252) * NXT-246 Indicator Old URL mixed casing - redirects not working correctly (#192) * NXT-246 Added middleware to convert all resolved urls to lowercase Next js is routing to different pages/components if the url uses mixed casing compared to lowercase. This fixes that. * NXT-246 Fixed check to compare mapped indicator url with resolved url * NXT-246 Fixed linting issues * NXT-246 Added unit test for middleware function * NXT-330 Indicators change chapters to chapter (#226) * change chapters to chapter in indicators [slug] and update unit tests * NXT-330 Removed obsolete snapshots --------- Co-authored-by: John Davey <john.davey@nice.org.uk> * NXT-337 Fix for empty product panel (#227) * NXT-337 Fix for missing _embedded.niceIndevPanel Undefined when no documents are present. * NXT-337 Further null checks for panel list * NXT-343 Exclude next-js manifest files from url lowercasing middlewar… (#228) * NXT-343 Exclude next-js manifest files from url lowercasing middleware and unit test. Co-authored-by: John Holland * NXT-343 Modify comment to trigger TC build * NXT-353 Remove middleware page (#235) * NXT-353 Remove middleware page * NXT-353 Reinstate error causing middleware files * NXT-242 Add Terraform config (#176) * NXT-210 Add test logs to status page * NXT-210 Move logging test into forms dir * NXT-242 Add Terraform config * NXT-242 Update vars * NXt-242 Change ip to non public * NXT-241-new k6 scripts - js files and json files * NXT-242 Change bucket name * NXT-242 Add values into json * NXT-242 Update tfvars * NXT-242 Change to octo vars * NXT-242 Update tf vars * NXT-242 Fix typo * NXT-242 Fix issues * NXT-242 Update tfvars * NXT-241-updates to perfomance test k6 scripts * NXT-241-revert changes to vscode settings * NXT-241-remove typeahead terms from combined scripts and cleanup * NXT-241-added more paths to the combined json file * NXT-241-edited galp_pages test data * NXT-242 Added load balancer config to tf * NXT-242 Add build number to config dir * NXT-242 Added diagnostic info * NXT-242 Update efs volume path * NXT-242 Update var names * NXT-242 Add node env * NXT-242 Refactor TF * NXT-242 Revert config.yml * NXT-242 Update default.yml * NXT-242 Remove comment * NXT-242 Variablise CPU and Memory * NXT-242 Remove breaking characters * NXT-242 Add variable characters * NXT-242 Change variable type to number * NXT-242 Move pino-mqrc.json * NXT-242 Fix pino-mq config problem * NXT-242 Fix issues with pino-mq * NXT-288 Enable AWS Exec # Conflicts: # aws/terraform/terraform.tfvars # aws/terraform/variables.tf * NXT-242 Add server count variable, remove cloudwatch variable * NXT-241 Add basic test and change dir * NXT-241-fix-merge-commits * NXT-241-add-more-test-data * NXT-241-add-stages-plus-the-noConnectionReuse * NXT-241-prettier-fixes * NXT-242 Add teamcity aws config * NXT-242 Add extra stuff to k6 testing file * NXT-347 Update testing files * NXT-242 Add gitignore * NXT-242 Add readme.md * NXT-350 Add curl to Docker * NXT-350 Add healthcheck * NXT-242 Remove PM2 * NXT-242 Update readme * NXT-210 Update logtest.page.tsx * NXT-242 Remove performance testing * NXT-242 Apply TF format * NXT-242 Fix typo * NXT-242 Add comments to readme * NXT-242 Update readme --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: FunmiOsho <60607637+FunmiOsho@users.noreply.github.com> Co-authored-by: lizaadebowale <liza.adebowale@nice.org.uk> Co-authored-by: L-Vazquez <37586672+L-Vazquez@users.noreply.github.com> * NXT-339 Update readme (#240) * NXT-353,NXT-343 404 errors on nextweb (#234) * NXT-343 Exclude next-js manifest files from url lowercasing middleware and unit test. Co-authored-by: John Holland * NXT-343 Modify comment to trigger TC build * NXT-343 Change url.pathname check to include _next/data and _next/static paths to avoid request errors * NXT-343 Add test for _next/data url condition * NXT-343 lowercase any mixed case indicators or guidance urls * NXT-343 Remove middleware page and unit tests * NXT-343 Hardcode indicators legacy redirects * NXT-343 Move indicators redirects from file directly to the config * NXT-343 Trying separate redirects file again * Revert "NXT-343 Trying separate redirects file again" This reverts commit f8d73685610fe09b773a9dd76bf3b26be8b8df01. * NXT-343 Correct redirects where there are duplicates with same case or missing lowercase version * Update package lock --------- Co-authored-by: John Holland <john.holland@nice.org.uk> Co-authored-by: John Davey <john.davey@nice.org.uk> Co-authored-by: John Davey <66066685+johndavey72@users.noreply.github.com> Co-authored-by: Colin M <12713935+colinmazu@users.noreply.github.com> Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: FunmiOsho <60607637+FunmiOsho@users.noreply.github.com> Co-authored-by: lizaadebowale <liza.adebowale@nice.org.uk> Co-authored-by: L-Vazquez <37586672+L-Vazquez@users.noreply.github.com> * NXT-345 Storyblok api key update (#239) * NXT-345: update initStoryblok to use new publicRunTime variable for SB. - add new yaml config variable - refactor catch all slug to logic in other pages Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * NXT-345: add test coverage for initStoryblok Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * NXT-345: add generic error as global const - update generic error message to global const - update tests to cover error handling Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * NXT-345: remove unused import * NXT-345 Linting fixes * NXT-345 update generic error message in SB util tests * NXT-345: remove server config properties and mocks --------- Co-authored-by: John Davey <johndavey72@users.noreply.github.com> Co-authored-by: John Davey <john.davey@nice.org.uk> * Regenerated package lock (#257) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Fix snapshot (#258) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-370 Add datetime to nextweb config dir name * NXT-374 remove middleware (#260) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-380 Update newsletter signup URL and related snapshots/references (#262) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-381 Rename chapters to chapter in indicator slug subdirectories for evidence and resources (#263) * NXT-382 Move hard-coded auth setting into config vars (#264) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NICE-306 Configure robots.txt for live & non-live environments (#265) * NICE-306 Configure robots.txt for live & non-live environments * NICE-306 Add denyRobots config var * NICE-306 robots txt api handler unit test * NXT-306 Refactor unit test for robots txt api Co-authored-by: Chris Barker <barkertron@users.noreply.github.com> * NICE-306 move jest.resetModules to beforeEach --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: John Davey <john.davey@nice.org.uk> Co-authored-by: Chris Barker <barkertron@users.noreply.github.com> * NXT-387 Fix fonts in global nav (#267) * NXT-387 Fix fonts in global nav * NXT-387 Update snapshots --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-386 Add new news redirects (#268) * NXT-386 Add new news redirects * NXT-386 Change redirect ordering --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: John Holland <john.holland@nice.org.uk> Co-authored-by: Claire Hunter <claire.hunter@nice.org.uk> Co-authored-by: John Davey <66066685+johndavey72@users.noreply.github.com> Co-authored-by: Dale Hudson <dalefrontend@gmail.com> Co-authored-by: John Davey <john.davey@nice.org.uk> Co-authored-by: John Davey <johndavey72@users.noreply.github.com> Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> Co-authored-by: Dale H <dale.hudson@nice.org.uk> Co-authored-by: Sue Garner <33061045+suegarner@users.noreply.github.com> Co-authored-by: Colin M <12713935+colinmazu@users.noreply.github.com> Co-authored-by: FunmiOsho <60607637+FunmiOsho@users.noreply.github.com> Co-authored-by: lizaadebowale <liza.adebowale@nice.org.uk> Co-authored-by: L-Vazquez <37586672+L-Vazquez@users.noreply.github.com> Co-authored-by: Chris Barker <barkertron@users.noreply.github.com>
lizaadebowale
added a commit
that referenced
this pull request
Oct 15, 2024
#289) * Nxt 180 scaffold pages (#131) * NXT-180 Add basic routing; refactor fonts * NXT-180 Add basic Storyblok integration * NXT-180 Display SB data on home and about pages * NXT-180 Fix TS issues, abstract Storyblok connection --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nxt 181 card components (#135) * NXT-181 Card grid homepage WIP * NXT-181 Fix TS errors * NXT-181 Add card component from NDS * NXT-181 TS integration for cards and homepage * NXT-181 Resolve Storyblok link objects * NXT-181 Fix links when SB returns empty link objects * NXT-181 Alter card grid layout, fix failing tests * NXT-181 Remove logging --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nxt 185 hero and catnav (#136) * NXT-185 Add hero and catnav components * NXT-185 Add SSL proxy for Storyblok preview mode * NXT-185 Resolve draft versions from context * NXT-185 Refactor page fetching, add catch-all about route * NXT-185 Breadcrumbs WIP --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-185 Add unit tests, fix broken page fetching * NXT-187 Refactor breadcrumb code (#137) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-188 corporate page metadata (#139) * NXT-188 Fix failing tests * NXT-188 Add metadata from Storyblok, update user-generated metadata * NXT-188 Add canonical and language link/meta tags --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Bump commit * Bump commit * Bump * Bump commit * Fix broken snapshot * Replace public config with server config for SB tokens * NXT-186 Static paths WIP (#153) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nxt 186 storyblok static paths (#155) * NXT-172 Add API endpoint for deleting fs cache entries (#122) * NXT-155 Change alert functionality to read from InDev * NXT-155 Updated snapshot test * NXT-155 Added alert logic to docs/consultation tabs for project page * NXT-155 Add API endpoint for deleting fs cache entries * NXT-155 Prevent cacheing of API cache deletion response * NXT-155 possible caching fixes (#134) * NXT-155 possible caching fixes * NXT-155 stop it generating multiple cache entries in different cases * NXT-155 move lowercase fixes to getCacheKey function * NXT-155 Lint cache api; send group keys in 400 err * NXT-155 fix tests --------- Co-authored-by: John Holland <john.holland@nice.org.uk> Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: Claire Hunter <claire.hunter@nice.org.uk> * NXT-178 Add international enquiry form and test (#132) * NXT-190 embedded doesnt exist for hub pages (#138) * NXT-186 Static paths WIP * NXT-186 Update linting rules --------- Co-authored-by: John Holland <john.holland@nice.org.uk> Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: Claire Hunter <claire.hunter@nice.org.uk> Co-authored-by: John Davey <66066685+johndavey72@users.noreply.github.com> * Nxt 186 storyblok static paths (#156) * NXT-172 Add API endpoint for deleting fs cache entries (#122) * NXT-155 Change alert functionality to read from InDev * NXT-155 Updated snapshot test * NXT-155 Added alert logic to docs/consultation tabs for project page * NXT-155 Add API endpoint for deleting fs cache entries * NXT-155 Prevent cacheing of API cache deletion response * NXT-155 possible caching fixes (#134) * NXT-155 possible caching fixes * NXT-155 stop it generating multiple cache entries in different cases * NXT-155 move lowercase fixes to getCacheKey function * NXT-155 Lint cache api; send group keys in 400 err * NXT-155 fix tests --------- Co-authored-by: John Holland <john.holland@nice.org.uk> Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: Claire Hunter <claire.hunter@nice.org.uk> * NXT-178 Add international enquiry form and test (#132) * NXT-190 embedded doesnt exist for hub pages (#138) * NXT-186 Static paths WIP * NXT-186 Update linting rules * NXT-186 Resolve custom rendering method --------- Co-authored-by: John Holland <john.holland@nice.org.uk> Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: Claire Hunter <claire.hunter@nice.org.uk> Co-authored-by: John Davey <66066685+johndavey72@users.noreply.github.com> * Add verbose logging to debug test env * Auto clear SB cache * NXT-211 Add simple server-side catch-all route (#162) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-197 Add breadcrumbs (#164) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Temporarily circumvent SB cache * Temp page header component update * Update Storyblok types; ignore generated file with ESLint * Correct linting setup * Make grid gutter loose * Emit logs from status page check * Update status page logs * Replace test status page errors with warnings * Fix lockfile * Update tsconfig settings * Add new SB types * WIP info page & nested rich text component * Nxt 229 news article page (#174) * chore: test updating the storyblok types from npm script * feat: test adding image to the storyblok hero and test using button as link if ctaLink and ctatext returned from Storyblok * chore: add storyblok richtext renderer as dep * feat: add NewsArticle component to render news articles in app - Add News Article component with PageHeader and richtext renderer for content. - Update PageHeader to render formatted date from design Dale H * style: test adding basic layout styles from scss * style: update image css * feat: first pass at blockquote we will use in the richtext examples in news articles * feat: test adding blockquote to the richtext renderer resolver * feat: first pass at NewsLetterSignup using DS components * styles: temp update to blockquote component style * feat: first pass at adding route for news with catch all for news-blogs-and-podcasts (TEMP till slug sturcture agreed) * chore: remove console log * refactor: update NewsArticle component to start using DS components * feat: first pass adding BlogPost component to render blog pages (TODO: move to its own route under pages dir) * refactor: remove empty h2 tag * feat: flesh out news article and move to correct path - Updated the news article to show panels on sidebar - updated StoryblokPageHeader to include tags and date (temporary test until new PageHeader version completed) - Added first pass on StoryblokImage component to use Storyblok image service, adds support for webp and jpeg formats Dale H * feat: first pass on sidebar panels for related news and links - Added first pass StoryblokRelatedLink and StoryblokRelatedNewsLink components - Added Panels to sidebar on NewsArticle Page TODO: update spacing across news article and it's components TODO: add tests for components and pages Dale H * refactor: update related news and link components to use Link component from next-web * chore: remove console.logs * feat: first pass at updating breadcrumbs - Update logic on breadcrumb function - Add path on getBreadcrumbs call, using partly hard coded path. TODO: get path dynamically, within getStaticProps Dale H * refactor: test PageHeader alpha release in StoryblokPageHeader * styles: update featuredImage offset styles to use custom property, so we can update the variable value inside of media queries * feat(ts): udpate type for image and pass rest prop to handle passing of common image attributes * refactor: update newsarticle image to take width and height and maintain 16/9 aspect ratio * test: update image component test and add fallback image for when src is empty * refactor: hardcode Breadcrumbs in news article. Comment out breadcrumb logic from StoryblokPageHeader to test news article * fix: remove updatedBreadcrumb * feat: start richtextRenderer component * styles: remove max width from blockquote to span full width of container * fix: revert StoryblokPageHeader component for stories that use the StoryblokPageHeader blok * feat: first pass on NewsPageHeader to use on news, in-depth, podcast and blog pages * feat: update richtext and news article components - Updated types for richtext component - Add NewsPageHeader and updated Richtext component to NewsArticle page component Dale H * styles: update style to handle images contained within richtext content * feat: first pass at YoutubeEmbed handled by rich text renderer * refactor: update newspageheader to use date util from repo * feat: Update richtext component to render inline images as StoryblokImage component ` * fix: add workaround to handle inline quotes in the richtext field of news stories NOTE: currently the children of the inline quote doesn't render correctly when using the blockquote component in the nodeResolver of storyblok-rich-text-renderer. We bump into a few issues around rendering and correct typing. I've hardcoded the html and imported styles so we can render the correctly styled quote TODO: Update the blockquote component or richtext types to handle passing through richtext quote children to the blockquote component * fix: update richtext interface, revert to initial interface from previous commit * tests: add tests for breadcrumbs and pageheader meta for tag and published date * fix: update breadcrumb name and resources link panel * test: remove test for fetchPriority attribute * refactor: update the inner Link component to use href over to attribute * tests: add relatednewslink tests * tests: first pass adding youtube embed tests TODO: add a field in storyblok for title so we can add an accessible title to the video embed Dale H * tests: firstpass adding tests for StoryblokRichText component * tests: first pass adding tests for Storyblok NewsArticle component * styles: update cta button so inner text wraps * fix: update types for newsarticle, add test to check whether signup action is rendered * tests: update NewsArticle tests * tests: first pass test news article slug page * fix: remove unused imports from richtext component and tests * chore: update design system version for new pageheader * refactor: update the newsarticle component to take new page header over newspageheader. - Update NewsArticle.tsx to use design system PageHeader new version - Add hardcoded breadcrumb values and handle in NewsArticle to be passed to PageHeader component TODO: Possible refactor of NewsPageHeader to be a generic component that can be used on news index and slug pages TODO: Image spacing to be handled via useEffect and ref to image to dynamically update the css varaible for its spacing Dale H ` * chore: rename pageheaderfooter that contains date and tag * refactor: update StoryblokImage and NewsArticle for overlap Added refs to NewsArticle and StoryblokImage so we can target them for updating the custom property value for the image overlap between header and body * style: update spacing around pageheader elements * tests: add further coverage on NewsArticle component * fix: udpate relatedlink and relatednewslink to use resolveStoyblokLink util function * round the pixels down on the featuredImageOffset value * fix: append /m/ for webp source image * fix: add title to Youtube embed for screen reader use * chore: update storyblok types * fix: update youtube iframe embed source value, remove comments * NICE-170 latest from nice & homepage (#175) * NICE-170 Homepage WIP * NICE-170 Latest news WIP * NICE-170 List latest news articles * NICE-170 Featured story WIP * NICE-170 Implement homepage hero * NICE-170 Action banner WIP * NICE-170 Resolved link, promo box WIP * NICE-170 Fix broken links for story objects * NICE-170 Complete "latest from NICE" section * NICE-170 Revert tsconfig, add NewsStory types * NICE-170 Add unit tests for homepage news section & components * NICE-170 Render news grid image as link, fix snapshots * NICE-170 Use better semantics, fix snapshots * NICE-170 misc fixes post review --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nxt 229 updates and fixes (#177) * fix: add empty dependancy array to useEffect to run once on mount, resize event for image * fix: add missing slash on jpeg src * feat: first pass adding author blok * styles: round image for author image * refactor: bring in update to fetchStory function and add author blok * first pass onblog post with listed authors - rough draft * update author component, extract the authorlist from blog post * update interface for blogpost and remove the inlined authorlist * fix: remove trailing comma from tsconfig * refactor: move authorlist to its own component, test author component * add authorList component and extend tests * start moving newspage header and child component into own files * fix: rename blogpost and newsarticle component to keep naming convention for Storyblok blok components * refactor: update AuthorList and tests * tests: start blogpost component tests * styles: update richtext elements vertical spacing and blog post * tests: update blog post slug page tests * tests: update youtube embed tests to include updated title property * tests: add missing slash from returned image source in tests * tests: update snapshot tests related to pageheader updates * styles: remove need for nested grid on BlogPost component, spacing updates on articles and posts * tests: update snapshots related to pageheader updates * tests: add testid to pageheader to test id applied, after pageheader updates * style: add commented styles to take over pageheader secondsection styles in NDS * feat: first pass at StoryblokIframe component * spike: spike nextjs/third-parties and react-lite youtube embeds - Tested next third parties, works but difficulty testing - Tested react-lite-youtube-embed, works and tested. TODO: test accessibility of react-lite-yt-embed further and against standard iframe implementation. TODO: ensure component is typed correctly * chore: remove spiked nextjs/third-parties * fix: remove incomplete import from richtext styles causing no styles to load * feat: use react-lite-youtube-embed for YT embed component * fix: update params and cookie props on on LiteYoutubeEmbed Issue with Safari not autoplaying the video once the poster play button was clicked. Adding cookie=true and updating params to include autoplay=1 solves this issue. * styles: update action banner spacing on newsarticle * styles: first pass normalise spacing to use DS spacing values * styles: normalise blockquote spacing * styles: tidy duplicated spacing * fix: add lodash debounce to handleResize functions on blog and news page components * tests: update breadcrumb test on slug page, and remove expect from youtubeembed test * NXT-250 Refactor fetchstories utility function (#179) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-251 Refactor NewsCard component (#180) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nxt 229 update tests and mocks (#181) * update test utils mocks for blog post and news article * fix: update type of blogpost blok and update tests to use mock util data * fix: udpate type of slug pages for blog and news to pass correct blok types * fix: udpate types and mock for newsarticle component - Update blok prop type and revert back to types generated by Storyblok - remove previous override type for news article - Update StoryblokNewsArticle to use mock data from test utils Dale H * refactor: update authorlist to render author blok over stoyblok component, and add className prop to AuthorBlok * tests: update test to check for author data in blogpost component * tests: update slug page tests on news and blog page slugs, using mock data from utils * tests: add richtext mock data and update tests that use richtext * NICE-169 secondary promo (#182) * NICE-169 Promo sections WIP * NICE-169 Complete Promo Box & tests --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nice 222 news blog style fixes (#184) * styles: update horizontal spacing * styles: update targetting on Youtube embed when child of richtext component * styles: update spacing on featuredImage and page header * refactor: normalise grid layout between newArticle and blogPost, update featuredImage margins * styles: switch featuredImage to 21/9 between sm and md breakpoints * styles: udpate spacing between main content/sidebar, header and action banner * styles: update bottom margin on youtube embed in richtext * styles: add fallback styles for inset and set button hover cursor to pointer * styles: update author styles from crit * styles: remove pageheader bottom margin * NICE-164 Implement Storyblok action banner (#185) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Big snapshot update * NICE-226 draft info page (#190) * NICE-226 Draft info page template * NICE-226 Revert Space ID * NICE-226 Remove log --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Fix linting error * Fix stylelint errors * Style fixes (#191) * fix: update styles to remove BEM convention in modules where we can * fix: add ignore on selector-class-pattern for targeting page header classes from DS in News and Blog components * Stylelint config update * NICE-178 Build the news listing template (#183) * NICE-178 Retrieve list of news articles from storyblok - wip * NICE-178 Add pagination component - wip * NICE-178 Return object with stories and total pages from storyblok response * NICE-178 Add current page and pagination destinations * NICE-178 Calculate total pages for pagination * NICE-178 Redirect to news/articles/ if page is outside bounds * NICE-178 Destructure gss props, return props directly * NICE-178 Componentise news list pagination * NICE-178 Rename prop as they aren't optional * NICE-178 Unit test newsListPagination wrapper * NICE-178 Revert tsconfig to fix issue with module resolution * NICE-178 Unit testing wip. Add npm script so space ID can be set and exported from terminal * NICE-178 Unit tests * NICE-178 Add action banner * NICE-178 Checkout web package.json etc to avoid merge conflict * NICE-178 Add NewsGrid component to articles list layout * NICE-178 Repurpose NewsGrid as NewsList for list pages * NICE-178 Move return in fetchStories to match storyblok-main * NICE-178 Add isListItem variant for news list. Setup basic style and layout for newslist page * NICE-178 Add featured story, sort stories by content.date descending. Update type useage * NICE-178 Add newslist nav component * NICE-178 Add npm script for updating types via shell * NICE-178 Update types * NICE-178 Add SEO * NICE-178 Adjust mobile styles for newscard * NICE-178 Add unit tests. Update snapshots * NICE-178 Add news nav to news index page. Change getActiveLink to match url paths exactly * NICE-178 Fix unit tests * NICE-178 Remove logging line * NICE-178 Move components to News dir to declutter main dir * NICE-178 Update location of pagination component * NICE-178 Prettify * NICE-178 Add unit tests for isNewsListItem variant * NICE-178 Comment out noisy router test * NICE-178 Skip first story on first page as it's featured. Articles index unit test - wip * NICE-178 Remove logger lines * NICE-178 Add todo for gssp on articles index * NICE-178 Unit tests * NICE-178 unit test gssp * NICE-178 remove worthless unit test * NICE-178 Update unit tests. Adjust border thickness for NewsCard * NICE-178 fix nesting level of selectors picked up by linting rule * NICE-178 Move storyblok utils mocks to outer describe * NICE-178 Remove magic number from unit test * NICE-178 Remove unnecessary string conversion in query * NICE-178 Handle storyblok fetchStories error condition * NICE-178 Revised error message * NICE-231 homepage update (#193) * NICE-231 Add homepage spacing, update promo box transparency * NICE-231 Complete Spotlight component * NICE-231 Update spotlight, add new news types * NICE-231 Fix rendering of homepage news links * NICE-231 Update snapshots * NICE-231 Update spotlight component to handle podcasts * NICE-231 Update snapshots, temporarily remove articles listing page unit tests * NICE-231 Fix TS errors * NICE-231 Homepage style updates * NICE-231 Fix snapshots --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Fix linting errors * Fix Card Grid link resolution * Nice 215 blog listing build (#195) * test: udpate snapshot test on newsarticle after updating mock data * udpate news and blog pages to filter scheduled stories according to date field * start validate route/pagination work * NICE-215 validateRouteParams wip * update types around validateRouteParams for storyblok * NICE-215 Handle no stories, parameterise validateRouteParams items and defaults * NICE-215 Fix starts_with after debug * test redirect and update blog page - first pass testing path redirect for out of range pagination pages - update blog listing page with new validateRouteParams - update news article listing page with resolvedUrl Authors: John Davey Dale Hudson * NICE-215 Remove debugging, tidy up out of range function * NICE-215 unit test featured story prop in gssp * NICE-215 pass sbParams differently, clean up types * NICE-215 Handle author content in FeaturedStory component * NICE-215 Change friendlyDate from toLocaleDateString to Intl.DateTimeFormat as this is more performant * NICE-215: update styles and destructure props for author in featuredStory * NICE-215: add first pass at author in newscard * NICE-215: update where we set AuthorHeadingElement * NICE-215: fix no rendering of secondSection if no authors length * NICE-215(fix): layout of author block * update storyblokimage for better handling of serviceOptions * NICE-215: update author card with iscardauthor and headinglevel props - update author to handle different heading levels with default to h4 - add prop to handle author when in card, updated styles for this too * NICE-215: update featured card with updated author component * remove redundant function and logs from SB utils * NICE-215(fix): update how author is handled in blogpost now author component updated * NICE-215(styles): udpate author styles in news and featuredStory cards * NICE-215: update full_slug link urls to absolute urls * NICE-215: udpate perf of background image and author keys - Added Storyblok image service parameters to resize and reduce quality of images to improve image loading times - Changed id for author lists within news and featured story cards * NICE-215: revert jpeg sources ready to refactor to use all image service options * NICE-215: add iframe component to storyblok init * NICE-215(fix): update image to ternary for correct loading of fallback image * NICE-215: fix Youtube embed button aria - remove announce prop from youtube embed, was duplicating aria label values - add fireevent to button click event in YT embed test * NICE-215(refactor): Update StoryblokImage component - Update image component to handle Storyblok image service options - As first pass we only cater for service options width, height, smart, quality - Update components using the StoryblokImage component with updated serviceOptions so we can optimise images currently in use. Dale H * NICE-215: handle error for empty stories array --------- Co-authored-by: John Davey <john.davey@nice.org.uk> Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * NICE-238 Add podcast page (#194) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * return null if author is string (#196) * Nice 215 fix tests (#197) * NICE-215: fix background images using SB image service by encoding brackets () * NICE-215: remove author as string from newsArticle as not part of content model * NICE-215(fix): Update snapshots for link and bgimg - updated snapshots for updated absolute url link handling in news and featured cards - updated snapshots for encoded background image urls containing storyblok image service filters - updated snapshots for featured and news cards with added authors John Davey Dale Hudson * fix pagination tests with updated prop names * NICE-215(fix): update snapshots from image component updates * NICE-215(fix): udpate tests after AuthorList refactor * NICE-215(fix): handle blogpost authors when array empty Handle blog posts author when they are pubilshed with an unpublished or no author attached. John D Dale H * Nice 231 homepage update (#198) * NICE-231 Add homepage spacing, update promo box transparency * NICE-231 Complete Spotlight component * NICE-231 Update spotlight, add new news types * NICE-231 Fix rendering of homepage news links * NICE-231 Update snapshots * NICE-231 Update spotlight component to handle podcasts * NICE-231 Update snapshots, temporarily remove articles listing page unit tests * NICE-231 Fix TS errors * NICE-231 Homepage style updates * NICE-231 Fix snapshots * NICE-231 Resolve merge conflicts * NICE-231 Homepage UI updates * NCIE-231 Limit news card to 1 author * NICE-231 Update author UI, change absolute URLs * NICE-231 Fix snapshots * NICE-231 Update Promobox mock data and snapshot --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nice 217 podcast listing page (#199) * NICE-217: podcast listing page, initial setup - Condtionally render newscard image - Update newscard author, return null if author is string D Hudson J Davey * NICE-217 Update podcast listing page - Make newsCard image optional - Fix broken unit test by removing redundant console log - Early return if author is just a string - Make newsListNav propless with destinations contained as they won't change - First pass at podcast listing page Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NICE-217 Add unit tests to cover optional showImage Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NICE-217 Update lead text on podcast list page * NICE-217: update breadcrumbs with absolute links --------- Co-authored-by: John Davey <john.davey@nice.org.uk> Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NICE-248 Resolve author linting issues (#200) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NICE-218 First pass at in-depth listing page (#201) Co-authored-by: John Davey <john.davey@nice.org.uk> Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * Nxt 270 news homepage (#202) * NXT-270 News homepage WIP * NXT-270 More news homepage WIP * NXT-270 Complete news homepage --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Fix linting issue * Add remote pattern image config * NICE 253 Refactor init storyblok (#204) * proof that separating initStoryblok enables proper mocking * streamline component imports for storyblokcomponent pages Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> --------- Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NICE-253 Fix issue with initStoryblok refactor breaking the catch all slug. Optimise component object for lean bundle. (#207) Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NXT-269(fix): filter articles out of latest news with future date (#208) * NICE-254 Handle image optimisation (#205) * NICE-254 Optimise images coming from storyblok image service. Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NICE-254 Update snapshots * NICE-254 Add optimise image util and unit test * NICE-254 Reinstate storyblokhero --------- Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * Nxt 292 blog page fixes from feedback (#209) * NXT-292: fix blog post author and author list headings * NXT-292: fix headings, not to render when no content * Merge storyblok-main Co-authored-by: John Davey <johndavey72@users.noreply.github.com> --------- Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * Nxt 292 blog page fixes from feedback (#210) * NXT-292: fix blog post author and author list headings * NXT-292: fix headings, not to render when no content * Merge storyblok-main Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * NXT-292 linting * NXT-292 merge sbmain --------- Co-authored-by: John Davey <johndavey72@users.noreply.github.com> Co-authored-by: John Davey <john.davey@nice.org.uk> * NXT-292: update homepage snapshot * NICE-255 Change enablejsapi from 0 to 1 (#212) * NICE-250 Managing focus on pagination route changes (#214) * spike pagination announcement for screenreader users - Add a div calling out the current page and total pages when navigating via screenreader - Update focus to hidden div, just above NewsList - this is one option of a few we need to test, preferably with screenreader users. - Options to go through linked on ticket: NICE-250 * NICE-250 Use Announcer component for pagination changes * NICE-250 add aria-hidden on focus element to prevent screen reader reading out 'blank' * NICE-250 Add announcer to other list pages. Pagination focus hook wip Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NICE-250 Roll out pagination focus to other list pages. Add component for pagination focus. * NICE-250 Remove console log * NICE-250 Add unit tests for pagination focus component * NICE-250 Refactor announcer code into a re-useable component * NICE-250 Remove unused deps and commented out code --------- Co-authored-by: dalecreativ <dalefrontend@gmail.com> Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NICE-260 Calculate canonical urls for slug pages and unit test (#216) Co-authored by Dale Hudson * Nxt 283 font size fixes (#215) * NXT-283: Update root font size to handle narrow viewport sizes. Co-authored by: John Davey * NXT-283: update font sizes for h1 on smaller screen sizes Targetting hero and page headings for home, news article, and blog post pages. Co-authored by: John Davey * NXT-283 Adjust h1 font-size for narrow viewport. Apply word-break and hyphenation for very narrow viewports Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> --------- Co-authored-by: John Davey <john.davey@nice.org.uk> Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * Nice 261 refactor image optimisation (#217) * NXT-261: refactor constructStoryBlokImageSrc to encodeParens and handle background images Co-authored by: John Davey. * NXT-261: null check image on podcast slig page * NXT-261: remove optimiseImage import from tests * NICE-261: update failing snapshot from encodeParens update (#218) * Nice 266 homepage spacing (#221) * NICE-266 Update homepage spacing; restore blog authors on news homepage * NICE-266 Miscellaneous QA fixes * NICE-266 Update blog page title * NICE-266 Set podcast image to unoptimised * NICE-266 Prevent future articles from appearing on news homepage * NICE-266 Pedantry --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nice 266 homepage spacing (#223) * NICE-266 Update homepage spacing; restore blog authors on news homepage * NICE-266 Miscellaneous QA fixes * NICE-266 Update blog page title * NICE-266 Set podcast image to unoptimised * NICE-266 Prevent future articles from appearing on news homepage * NICE-266 Pedantry * NICE-266 Set base URL for podcast image --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nice 266 homepage spacing (#224) * NICE-266 Update homepage spacing; restore blog authors on news homepage * NICE-266 Miscellaneous QA fixes * NICE-266 Update blog page title * NICE-266 Set podcast image to unoptimised * NICE-266 Prevent future articles from appearing on news homepage * NICE-266 Pedantry * NICE-266 Set base URL for podcast image * NICE-266 Pluralise podcasts label --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NICE-283 spacing again (#229) * NICE-283 Make spacing and content changes * NICE-283 Spacing, routing, text updates * NICE-283 A11y fixes * NICE-238 Amend news article action banner spacing * NICE-283 Update action banner spacing * NICE-283 Restore access token config --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Update snapshots * NXT-276 Update default image with base URL from config * NXT-217 Use Ocelot for Storyblok cacheing (#225) * NXT-217 Installed Ocelot.Administration NuGet package * NXT-217 Added Storyblok config to Ocelot routes * NXT-217 Configure Storyblok to use Ocelot * NXT-217 Add Ocelot Administration endpoint * NXT-217 Refactor Ocelot cache config settings * NXT-217 Add Ocelot for Storyblok section to readme * NXT-217 Update Readme --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NICE-291 Return 404 for root catch all unless env var enables it (#230) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-346 Revert env var config, add test page (#232) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nxt 346 revert env vars (#233) * NXT-346 Revert env var config, add test page * NXT-346 Add test env var to config --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * fix: update default ocelotEndpoint to empty value * Remove env var from next config * NXT-319, NICE-251 unit testing utility functions and news index pages, adding response status from SB API (#231) * initial commit * add boilerplate for get server side props * update validaterouteparams to return redirect if page is less than 1 TODO: alternative approach of assigning page to 1 if query.page is less than 1 * update mock data and refactor mocks for listing page * stub out getStoryblokAPi get method and modify response per test * Added json files for various responses Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * Added further test for fetchStory util co-authored by Dale Hudson * NXT-319 Add server error unit test Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NXT-319 Add unit tests for fetchStories co-authored by Dale Hudson * NXT-319 Add fetchLinks unit tests Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NXT-319 Add fetchLinks unit tests Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NXT-319 Remove promise.reject from fetchStories to prevent unreachable code issue. Tidy up imports in storyblok utils. Co-authored by Dale Hudson * NXT-319 Change mock links success response to match item returned from storyblok/react. Add note to investigate jest setup mocking issue * NXT-319 Add getBreadcrumbs unit tests * NXT-319 Add error handling to friendlyDate. Unit test friendlyDate * NXT-301: add test coverage for getNewsTypes * NXT-319: update validateRouteParams query.page to be set to one if less than 1 * NXT-319: start validateRouteParams test coverage * NXT-319: update news index page tests after updates * NXT-319: test for success and error responses from fetchStories returned to valiudateRouteParams * NXt-319: remove skip test validateRouteParam * NXT-319 Handle no stories on index pages. Return not founds for incorrect query page param. Refactor and add unit tests for validateRouteParams. Remove log lines. Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NXT-319 Remove unused spy. Add unit test todos for news articles listing" * NXT-319 Add unit test to check undefined return from getSlugFromParams. Remove unused dep * NXT-319 Add unit tests for news article list page. Refactor around thrown errors from validateRouteParams. Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NXT-319 Unit tests of news articles renderins. Gssp correct props unit test. * NXT-319 Rendering unit tests for news articles listing Co-authored by: Dale Hudson * NXT-319 Convert in-depth gssp to try catch format. Adjust types to separate error props from success props. * NXT-319 Move isError util function to storyblok utils and unit test. Make public facing error messages more generic". Update snapshot. * NXT-319 Update blog posts index gssp to try catch format. Separate error and success props and types" * NXT-319: update podcast gssp to handle errors from api * NXT-319 Remove resolved url from gssp args. Update unit tests to reflect removal of API details in error messages. * NXT-319 Add unit tests for news index pages * NXT-319 Add further unit tests for news index pages. * NXT-319 Refactor blog slug page. Added blog slug page unit tests. Added mock success response for blog post. * NXT-319 Refine types for fetchStory, blog slug page and unit test mock data types. Remove error property from fetchStories. Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NXT-319 Remove unused deps * NXT-319: remove test backup files * NXT-319: refactor news article slug with TODO comments for discussion * NXT-319: add test coverage for articles slug page * NXT-319: add TODO for discussion on fetchStory * NXT-319: add snapshot for news article page * NXT-319: remove redundant imports from news slug tests * NXT-319: add mockData for podcast page * NXT-319: refactor podcast slug page in line with other slug pages * NXT-319: add tests for podcast slug page with snapshot * NXT-319: udpate type for newsarticle fetch and props. Add logger on getadditionalMetaTag failure * NXT-319: add useMemo back to getAdditionalMetatags in blog post page * NXT-319: add logger to getAdditionalMetaTags in podcast page * NXT-319: remove unused imports from news slug page * Add further test coverage for podcast slug page - updated snapshots Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * NXT-319: remove redundant mocks from blog slug page tests * NXT-319 Error handling for news index and unit test - wip * NXT-319: test news homepage, gssp and error handling * NXT-319: add tests for homepage and handle fetch errors from API. Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * NXT-319: fix linting errors Co-authored by: John Davey * NXT-319: update error handling in utils. consistant messaging and var naming * NXT-319: remove TODOs * NXT-319: update homepage snapshot and test with error message update Co-authored by: John Davey * NXT-319: replace isError from utils with lodash isError equivalent Co-authored by: John Davey * NXT-319: remove isError import from test * NXT-319: move TODOs to tech debt and remove redundant comments. - update podcast listing mock data Co-authored by: John Davey --------- Co-authored-by: John Davey <john.davey@nice.org.uk> Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * Nxt 301 blog page fixes from feedback (#219) * NXT-301: order list of authors alphabetically * NXT-301: add border and spacing to author section in blog header * NXT-301: update author section spacing and remove bottom margin from any richtext component last child element * NXT-301: remove TODOs that have been resolved through updates or discussions * NXT-301: increase on author name if img is included * NXT-301: add role presentation to images carrying no alt text * NXT-301: update width of PageHeader fullwidth-container Full width container updates changed the alignment of pageheader to the rest of content on blog and news articles. Updated so content between header and main content is aligned. * NXT-301: add stylelint ignore rule for global styles targetting page-header * NXT-301: update author blok border with token for colour * NXT-301: revert page header container horizontal spacing. * NXT-301: pageheader author block spacing - Only onclude border and bottom padding over breakpoint * NXT-301: first pass at updating the richtext image aspect ratios based on storyblok image width and heights * NXT-301: update test data to have accurate url structure for SB CDN images * NXT-301: update displayed dates to use the time tag. - update components that display dates to use time tage - update the mock data for dates - refactor the news/blog header meta to use getNewsType util TODO: refactor tag and date into meta component for reuse across other components Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * NXT-301: update richtext image - update aspect ratio, width and height attrs using SB image url path to get width and height * NXT-301: replace storyblok cdn and link domain from url mock data. Mitigate calls to actual url. * NXT-301: upate storyblok-data mock urls * NXT-301: update snapshots after mock data changes to urls * NXT-301: update styles to nudge featured image - Nudge pixels on page header and featuredimage across news and blog page templates - Keeps margin top within 2rem of page header bottom content TODO: revisit handling of featuredImage offsets. Adding to tech debt for post go live Dale H * NXT-301: remove TODOs and put in tech debt log for discussion * NXT-301: move setComponent causing rendering issues --------- Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * fix: add back lint disable line for global targetting of css from DS * Main merge (#242) * NXT-246 Indicator Old URL mixed casing - redirects not working correctly (#192) * NXT-246 Added middleware to convert all resolved urls to lowercase Next js is routing to different pages/components if the url uses mixed casing compared to lowercase. This fixes that. * NXT-246 Fixed check to compare mapped indicator url with resolved url * NXT-246 Fixed linting issues * NXT-246 Added unit test for middleware function * NXT-330 Indicators change chapters to chapter (#226) * change chapters to chapter in indicators [slug] and update unit tests * NXT-330 Removed obsolete snapshots --------- Co-authored-by: John Davey <john.davey@nice.org.uk> * NXT-337 Fix for empty product panel (#227) * NXT-337 Fix for missing _embedded.niceIndevPanel Undefined when no documents are present. * NXT-337 Further null checks for panel list * NXT-343 Exclude next-js manifest files from url lowercasing middlewar… (#228) * NXT-343 Exclude next-js manifest files from url lowercasing middleware and unit test. Co-authored-by: John Holland * NXT-343 Modify comment to trigger TC build * NXT-353 Remove middleware page (#235) * NXT-353 Remove middleware page * NXT-353 Reinstate error causing middleware files * NXT-242 Add Terraform config (#176) * NXT-210 Add test logs to status page * NXT-210 Move logging test into forms dir * NXT-242 Add Terraform config * NXT-242 Update vars * NXt-242 Change ip to non public * NXT-241-new k6 scripts - js files and json files * NXT-242 Change bucket name * NXT-242 Add values into json * NXT-242 Update tfvars * NXT-242 Change to octo vars * NXT-242 Update tf vars * NXT-242 Fix typo * NXT-242 Fix issues * NXT-242 Update tfvars * NXT-241-updates to perfomance test k6 scripts * NXT-241-revert changes to vscode settings * NXT-241-remove typeahead terms from combined scripts and cleanup * NXT-241-added more paths to the combined json file * NXT-241-edited galp_pages test data * NXT-242 Added load balancer config to tf * NXT-242 Add build number to config dir * NXT-242 Added diagnostic info * NXT-242 Update efs volume path * NXT-242 Update var names * NXT-242 Add node env * NXT-242 Refactor TF * NXT-242 Revert config.yml * NXT-242 Update default.yml * NXT-242 Remove comment * NXT-242 Variablise CPU and Memory * NXT-242 Remove breaking characters * NXT-242 Add variable characters * NXT-242 Change variable type to number * NXT-242 Move pino-mqrc.json * NXT-242 Fix pino-mq config problem * NXT-242 Fix issues with pino-mq * NXT-288 Enable AWS Exec # Conflicts: # aws/terraform/terraform.tfvars # aws/terraform/variables.tf * NXT-242 Add server count variable, remove cloudwatch variable * NXT-241 Add basic test and change dir * NXT-241-fix-merge-commits * NXT-241-add-more-test-data * NXT-241-add-stages-plus-the-noConnectionReuse * NXT-241-prettier-fixes * NXT-242 Add teamcity aws config * NXT-242 Add extra stuff to k6 testing file * NXT-347 Update testing files * NXT-242 Add gitignore * NXT-242 Add readme.md * NXT-350 Add curl to Docker * NXT-350 Add healthcheck * NXT-242 Remove PM2 * NXT-242 Update readme * NXT-210 Update logtest.page.tsx * NXT-242 Remove performance testing * NXT-242 Apply TF format * NXT-242 Fix typo * NXT-242 Add comments to readme * NXT-242 Update readme --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: FunmiOsho <60607637+FunmiOsho@users.noreply.github.com> Co-authored-by: lizaadebowale <liza.adebowale@nice.org.uk> Co-authored-by: L-Vazquez <37586672+L-Vazquez@users.noreply.github.com> * Update package lock --------- Co-authored-by: John Holland <john.holland@nice.org.uk> Co-authored-by: John Davey <john.davey@nice.org.uk> Co-authored-by: John Davey <66066685+johndavey72@users.noreply.github.com> Co-authored-by: Colin M <12713935+colinmazu@users.noreply.github.com> Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: FunmiOsho <60607637+FunmiOsho@users.noreply.github.com> Co-authored-by: lizaadebowale <liza.adebowale@nice.org.uk> Co-authored-by: L-Vazquez <37586672+L-Vazquez@users.noreply.github.com> * Nice 249 sitemap generation (#245) * NICE-249 Fetch news content for sitemap from Links API, unignore sitemaps from Git, edit static sitemap from Orchard * NICE-249 Complete sitemap generation * NICE-249 Scrub auto-generated sitemap --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-345 Consolidate preview and public access tokens, remove env test page (#246) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-369 Add redirects for news section (#247) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Remove cv parameter from sbParams if Ocelot not present * Fix incorrect news redirect * Fix failing test due to CV parameter * Storyblok merge main (#250) * NXT-246 Indicator Old URL mixed casing - redirects not working correctly (#192) * NXT-246 Added middleware to convert all resolved urls to lowercase Next js is routing to different pages/components if the url uses mixed casing compared to lowercase. This fixes that. * NXT-246 Fixed check to compare mapped indicator url with resolved url * NXT-246 Fixed linting issues * NXT-246 Added unit test for middleware function * NXT-330 Indicators change chapters to chapter (#226) * change chapters to chapter in indicators [slug] and update unit tests * NXT-330 Removed obsolete snapshots --------- Co-authored-by: John Davey <john.davey@nice.org.uk> * NXT-337 Fix for empty product panel (#227) * NXT-337 Fix for missing _embedded.niceIndevPanel Undefined when no documents are present. * NXT-337 Further null checks for panel list * NXT-343 Exclude next-js manifest files from url lowercasing middlewar… (#228) * NXT-343 Exclude next-js manifest files from url lowercasing middleware and unit test. Co-authored-by: John Holland * NXT-343 Modify comment to trigger TC build * NXT-353 Remove middleware page (#235) * NXT-353 Remove middleware page * NXT-353 Reinstate error causing middleware files * NXT-242 Add Terraform config (#176) * NXT-210 Add test logs to status page * NXT-210 Move logging test into forms dir * NXT-242 Add Terraform config * NXT-242 Update vars * NXt-242 Change ip to non public * NXT-241-new k6 scripts - js files and json files * NXT-242 Change bucket name * NXT-242 Add values into json * NXT-242 Update tfvars * NXT-242 Change to octo vars * NXT-242 Update tf vars * NXT-242 Fix typo * NXT-242 Fix issues * NXT-242 Update tfvars * NXT-241-updates to perfomance test k6 scripts * NXT-241-revert changes to vscode settings * NXT-241-remove typeahead terms from combined scripts and cleanup * NXT-241-added more paths to the combined json file * NXT-241-edited galp_pages test data * NXT-242 Added load balancer config to tf * NXT-242 Add build number to config dir * NXT-242 Added diagnostic info * NXT-242 Update efs volume path * NXT-242 Update var names * NXT-242 Add node env * NXT-242 Refactor TF * NXT-242 Revert config.yml * NXT-242 Update default.yml * NXT-242 Remove comment * NXT-242 Variablise CPU and Memory * NXT-242 Remove breaking characters * NXT-242 Add variable characters * NXT-242 Change variable type to number * NXT-242 Move pino-mqrc.json * NXT-242 Fix pino-mq config problem * NXT-242 Fix issues with pino-mq * NXT-288 Enable AWS Exec # Conflicts: # aws/terraform/terraform.tfvars # aws/terraform/variables.tf * NXT-242 Add server count variable, remove cloudwatch variable * NXT-241 Add basic test and change dir * NXT-241-fix-merge-commits * NXT-241-add-more-test-data * NXT-241-add-stages-plus-the-noConnectionReuse * NXT-241-prettier-fixes * NXT-242 Add teamcity aws config * NXT-242 Add extra stuff to k6 testing file * NXT-347 Update testing files * NXT-242 Add gitignore * NXT-242 Add readme.md * NXT-350 Add curl to Docker * NXT-350 Add healthcheck * NXT-242 Remove PM2 * NXT-242 Update readme * NXT-210 Update logtest.page.tsx * NXT-242 Remove performance testing * NXT-242 Apply TF format * NXT-242 Fix typo * NXT-242 Add comments to readme * NXT-242 Update readme --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: FunmiOsho <60607637+FunmiOsho@users.noreply.github.com> Co-authored-by: lizaadebowale <liza.adebowale@nice.org.uk> Co-authored-by: L-Vazquez <37586672+L-Vazquez@users.noreply.github.com> * NXT-339 Update readme (#240) * NXT-353,NXT-343 404 errors on nextweb (#234) * NXT-343 Exclude next-js manifest files from url lowercasing middleware and unit test. Co-authored-by: John Holland * NXT-343 Modify comment to trigger TC build * NXT-343 Change url.pathname check to include _next/data and _next/static paths to avoid request errors * NXT-343 Add test for _next/data url condition * NXT-343 lowercase any mixed case indicators or guidance urls * NXT-343 Remove middleware page and unit tests * NXT-343 Hardcode indicators legacy redirects * NXT-343 Move indicators redirects from file directly to the config * NXT-343 Trying separate redirects file again * Revert "NXT-343 Trying separate redirects file again" This reverts commit f8d73685610fe09b773a9dd76bf3b26be8b8df01. * NXT-343 Correct redirects where there are duplicates with same case or missing lowercase version * Update package lock --------- Co-authored-by: John Holland <john.holland@nice.org.uk> Co-authored-by: John Davey <john.davey@nice.org.uk> Co-authored-by: John Davey <66066685+johndavey72@users.noreply.github.com> Co-authored-by: Colin M <12713935+colinmazu@users.noreply.github.com> Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: FunmiOsho <60607637+FunmiOsho@users.noreply.github.com> Co-authored-by: lizaadebowale <liza.adebowale@nice.org.uk> Co-authored-by: L-Vazquez <37586672+L-Vazquez@users.noreply.github.com> * Storyblok main merge conflicts (#252) * NXT-246 Indicator Old URL mixed casing - redirects not working correctly (#192) * NXT-246 Added middleware to convert all resolved urls to lowercase Next js is routing to different pages/components if the url uses mixed casing compared to lowercase. This fixes that. * NXT-246 Fixed check to compare mapped indicator url with resolved url * NXT-246 Fixed linting issues * NXT-246 Added unit test for middleware function * NXT-330 Indicators change chapters to chapter (#226) * change chapters to chapter in indicators [slug] and update unit tests * NXT-330 Removed obsolete snapshots --------- Co-authored-by: John Davey <john.davey@nice.org.uk> * NXT-337 Fix for empty product panel (#227) * NXT-337 Fix for missing _embedded.niceIndevPanel Undefined when no documents are present. * NXT-337 Further null checks for panel list * NXT-343 Exclude next-js manifest files from url lowercasing middlewar… (#228) * NXT-343 Exclude next-js manifest files from url lowercasing middleware and unit test. Co-authored-by: John Holland * NXT-343 Modify comment to trigger TC build * NXT-353 Remove middleware page (#235) * NXT-353 Remove middleware page * NXT-353 Reinstate error causing middleware files * NXT-242 Add Terraform config (#176) * NXT-210 Add test logs to status page * NXT-210 Move logging test into forms dir * NXT-242 Add Terraform config * NXT-242 Update vars * NXt-242 Change ip to non public * NXT-241-new k6 scripts - js files and json files * NXT-242 Change bucket name * NXT-242 Add values into json * NXT-242 Update tfvars * NXT-242 Change to octo vars * NXT-242 Update tf vars * NXT-242 Fix typo * NXT-242 Fix issues * NXT-242 Update tfvars * NXT-241-updates to perfomance test k6 scripts * NXT-241-revert changes to vscode settings * NXT-241-remove typeahead terms from combined scripts and cleanup * NXT-241-added more paths to the combined json file * NXT-241-edited galp_pages test data * NXT-242 Added load balancer config to tf * NXT-242 Add build number to config dir * NXT-242 Added diagnostic info * NXT-242 Update efs volume path * NXT-242 Update var names * NXT-242 Add node env * NXT-242 Refactor TF * NXT-242 Revert config.yml * NXT-242 Update default.yml * NXT-242 Remove comment * NXT-242 Variablise CPU and Memory * NXT-242 Remove breaking characters * NXT-242 Add variable characters * NXT-242 Change variable type to number * NXT-242 Move pino-mqrc.json * NXT-242 Fix pino-mq config problem * NXT-242 Fix issues with pino-mq * NXT-288 Enable AWS Exec # Conflicts: # aws/terraform/terraform.tfvars # aws/terraform/variables.tf * NXT-242 Add server count variable, remove cloudwatch variable * NXT-241 Add basic test and change dir * NXT-241-fix-merge-commits * NXT-241-add-more-test-data * NXT-241-add-stages-plus-the-noConnectionReuse * NXT-241-prettier-fixes * NXT-242 Add teamcity aws config * NXT-242 Add extra stuff to k6 testing file * NXT-347 Update testing files * NXT-242 Add gitignore * NXT-242 Add readme.md * NXT-350 Add curl to Docker * NXT-350 Add healthcheck * NXT-242 Remove PM2 * NXT-242 Update readme * NXT-210 Update logtest.page.tsx * NXT-242 Remove performance testing * NXT-242 Apply TF format * NXT-242 Fix typo * NXT-242 Add comments to readme * NXT-242 Update readme --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: FunmiOsho <60607637+FunmiOsho@users.noreply.github.com> Co-authored-by: lizaadebowale <liza.adebowale@nice.org.uk> Co-authored-by: L-Vazquez <37586672+L-Vazquez@users.noreply.github.com> * NXT-339 Update readme (#240) * NXT-353,NXT-343 404 errors on nextweb (#234) * NXT-343 Exclude next-js manifest files from url lowercasing middleware and unit test. Co-authored-by: John Holland * NXT-343 Modify comment to trigger TC build * NXT-343 Change url.pathname check to include _next/data and _next/static paths to avoid request errors * NXT-343 Add test for _next/data url condition * NXT-343 lowercase any mixed case indicators or guidance urls * NXT-343 Remove middleware page and unit tests * NXT-343 Hardcode indicators legacy redirects * NXT-343 Move indicators redirects from file directly to the config * NXT-343 Trying separate redirects file again * Revert "NXT-343 Trying separate redirects file again" This reverts commit f8d73685610fe09b773a9dd76bf3b26be8b8df01. * NXT-343 Correct redirects where there are duplicates with same case or missing lowercase version * Update package lock --------- Co-authored-by: John Holland <john.holland@nice.org.uk> Co-authored-by: John Davey <john.davey@nice.org.uk> Co-authored-by: John Davey <66066685+johndavey72@users.noreply.github.com> Co-authored-by: Colin M <12713935+colinmazu@users.noreply.github.com> Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: FunmiOsho <60607637+FunmiOsho@users.noreply.github.com> Co-authored-by: lizaadebowale <liza.adebowale@nice.org.uk> Co-authored-by: L-Vazquez <37586672+L-Vazquez@users.noreply.github.com> * NXT-345 Storyblok api key update (#239) * NXT-345: update initStoryblok to use new publicRunTime variable for SB. - add new yaml config variable - refactor catch all slug to logic in other pages Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * NXT-345: add test coverage for initStoryblok Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * NXT-345: add generic error as global const - update generic error message to global const - update tests to cover error handling Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * NXT-345: remove unused import * NXT-345 Linting fixes * NXT-345 update generic error message in SB util tests * NXT-345: remove server config properties and mocks --------- Co-authored-by: John Davey <johndavey72@users.noreply.github.com> Co-authored-by: John Davey <john.davey@nice.org.uk> * Regenerated package lock (#257) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Fix snapshot (#258) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-370 Add datetime to nextweb config dir name * NXT-373 Fix incorrect logger error component names * NXT-373 Add verbose logging for fetchStory Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NXT-374 remove middleware (#260) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-373 Fix unnecessary import * NXT-373 Add further information to logger error, change structure of logger error message Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NXT-373: update fetchStory and initStoryblok with info logs to test kibana log titles * NXT-373: comment out logger.info in initStoryblok logger.info and tests causing knock on issue in other tests e.g. CopytoClipboard component TODO: look at alternatives to mocking or spying on logger that don't cause issues across app * NXT-373: update mocks for logger to fix failing tests. * NXT-373: switch order of logger args. - add req headers to logs for test output in kibana - update tests to include empty req headers for passing tests. - update order of args being passed to logger TODO: check mocks for req headers if required * NXT-380 Update newsletter signup URL and related snapshots/references (#262) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-373: update info logs and add log to catch all slug * NXT-381 Rename chapters to chapter in indicator slug subdirectories for evidence and resources (#263) * NXT-382 Move hard-coded auth setting into config vars (#264) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-373 recreate package-lock after merge * NXT-373 Remove missed merge conflict markers * NXT-373 Update snapshots * NXT-373 Reinstate req object to context in tests * NXT-373 Bring next.config.js across from main * NXT-373 Add null checks and slight refactor of flattenNavigators to eliminate noise in error logs * NXT-373 roll out error logging from news articles to remaining slug pages. - udpated shape of fetchStories logging - brought news section slug pages in line with news slug page Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * NXT-373 (fix): don't throw on invalid date pass invalid date as string * NXT-373 (refactor): use mocks over spies for logger service - avoid any actual calls to logger service and replace with logger mock - move mock to setup file - replace spyon with mock in storyblok utils and storyblokInit tests * NXT-373 : update logger mock JotForm test to include logger.warn mock * NXT-373 test warn log on catch all slug page * NXT-373 : test logs around authors for caching * NXT-373 - update storyblok/react library to latest and refactor types to conform with update - updating storyblok/react library resolves caching issues around resolve_relations. - types for SB stories required updating to ISbStoryData after storyblok/react bump to latest. * NXT-373 refactor error handling and logging on fetchStory Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * NXT-373 update error handling and logging in fetchStories and validateRouteParam - update error hadnling in fetchStories, - pass error cause through from SB API and log as orginatingError - update tests on fetchStories and ValidateRouteParams * NXT-373: add logging to gssp on blog index page - logging originatingError as error.cause passed up from API call - update tests on blog index to cover logging * NXT-373: udpate blog slug page tests - update tests to cover logging on 404 and errors returned from fetchStory * NXT-373 add error logs with cause from API to news section pages - update logging on index and slig pages for news section - pass error cause up from storyblok fetch utils to pages to trace errors - update tests to include coverage of logging * NXT-327 update site_name on og metadata * NXT-373 update logging properties to be consistent across utils and pages. * NXT-373 remove logger warn for context check in catchallSlug * NXT-373 add error log and logger test for homepage * NXT-373 Update sb types * NXT-373 update news landing page error logging. * NXT-373 update storyblok type generation to use storyblok cli - Update deps and scripts to use storyblok cli for type generation - uninstall 3rd party dep storyblok-generate-ts - Update StoryblokStory types to ISbStoryData * NXT-373 update mock data to test author data in blog listing Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * NXT-373 resolve PR feedback for comments and imports * NXT-373 fix update SEO test after text change * NXT-373 Simplify error messages so they can reach kibana --------- Co-authored-by: Chris Barker <chris@christopherbarker.co.uk> Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: John Holland <john.holland@nice.org.uk> Co-authored-by: Claire Hunter <claire.hunter@nice.org.uk> Co-authored-by: Dale Hudson <dalefrontend@gmail.com> Co-authored-by: John Davey <johndavey72@users.noreply.github.com> Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> Co-authored-by: Dale H <dale.hudson@nice.org.uk> Co-authored-by: Sue Garner <33061045+suegarner@users.noreply.github.com> Co-authored-by: Colin M <12713935+colinmazu@users.noreply.github.com> Co-authored-by: FunmiOsho <60607637+FunmiOsho@users.noreply.github.com> Co-authored-by: lizaadebowale <liza.adebowale@nice.org.uk> Co-authored-by: L-Vazquez <37586672+L-Vazquez@users.noreply.github.com>
FunmiOsho
added a commit
that referenced
this pull request
Oct 21, 2024
* Nxt 180 scaffold pages (#131) * NXT-180 Add basic routing; refactor fonts * NXT-180 Add basic Storyblok integration * NXT-180 Display SB data on home and about pages * NXT-180 Fix TS issues, abstract Storyblok connection --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nxt 181 card components (#135) * NXT-181 Card grid homepage WIP * NXT-181 Fix TS errors * NXT-181 Add card component from NDS * NXT-181 TS integration for cards and homepage * NXT-181 Resolve Storyblok link objects * NXT-181 Fix links when SB returns empty link objects * NXT-181 Alter card grid layout, fix failing tests * NXT-181 Remove logging --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nxt 185 hero and catnav (#136) * NXT-185 Add hero and catnav components * NXT-185 Add SSL proxy for Storyblok preview mode * NXT-185 Resolve draft versions from context * NXT-185 Refactor page fetching, add catch-all about route * NXT-185 Breadcrumbs WIP --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-185 Add unit tests, fix broken page fetching * NXT-187 Refactor breadcrumb code (#137) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-188 corporate page metadata (#139) * NXT-188 Fix failing tests * NXT-188 Add metadata from Storyblok, update user-generated metadata * NXT-188 Add canonical and language link/meta tags --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Bump commit * Bump commit * Bump * Bump commit * Fix broken snapshot * Replace public config with server config for SB tokens * NXT-186 Static paths WIP (#153) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nxt 186 storyblok static paths (#155) * NXT-172 Add API endpoint for deleting fs cache entries (#122) * NXT-155 Change alert functionality to read from InDev * NXT-155 Updated snapshot test * NXT-155 Added alert logic to docs/consultation tabs for project page * NXT-155 Add API endpoint for deleting fs cache entries * NXT-155 Prevent cacheing of API cache deletion response * NXT-155 possible caching fixes (#134) * NXT-155 possible caching fixes * NXT-155 stop it generating multiple cache entries in different cases * NXT-155 move lowercase fixes to getCacheKey function * NXT-155 Lint cache api; send group keys in 400 err * NXT-155 fix tests --------- Co-authored-by: John Holland <john.holland@nice.org.uk> Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: Claire Hunter <claire.hunter@nice.org.uk> * NXT-178 Add international enquiry form and test (#132) * NXT-190 embedded doesnt exist for hub pages (#138) * NXT-186 Static paths WIP * NXT-186 Update linting rules --------- Co-authored-by: John Holland <john.holland@nice.org.uk> Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: Claire Hunter <claire.hunter@nice.org.uk> Co-authored-by: John Davey <66066685+johndavey72@users.noreply.github.com> * Nxt 186 storyblok static paths (#156) * NXT-172 Add API endpoint for deleting fs cache entries (#122) * NXT-155 Change alert functionality to read from InDev * NXT-155 Updated snapshot test * NXT-155 Added alert logic to docs/consultation tabs for project page * NXT-155 Add API endpoint for deleting fs cache entries * NXT-155 Prevent cacheing of API cache deletion response * NXT-155 possible caching fixes (#134) * NXT-155 possible caching fixes * NXT-155 stop it generating multiple cache entries in different cases * NXT-155 move lowercase fixes to getCacheKey function * NXT-155 Lint cache api; send group keys in 400 err * NXT-155 fix tests --------- Co-authored-by: John Holland <john.holland@nice.org.uk> Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: Claire Hunter <claire.hunter@nice.org.uk> * NXT-178 Add international enquiry form and test (#132) * NXT-190 embedded doesnt exist for hub pages (#138) * NXT-186 Static paths WIP * NXT-186 Update linting rules * NXT-186 Resolve custom rendering method --------- Co-authored-by: John Holland <john.holland@nice.org.uk> Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: Claire Hunter <claire.hunter@nice.org.uk> Co-authored-by: John Davey <66066685+johndavey72@users.noreply.github.com> * Add verbose logging to debug test env * Auto clear SB cache * NXT-211 Add simple server-side catch-all route (#162) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-197 Add breadcrumbs (#164) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Temporarily circumvent SB cache * Temp page header component update * Update Storyblok types; ignore generated file with ESLint * Correct linting setup * Make grid gutter loose * Emit logs from status page check * Update status page logs * Replace test status page errors with warnings * Fix lockfile * Update tsconfig settings * Add new SB types * WIP info page & nested rich text component * Nxt 229 news article page (#174) * chore: test updating the storyblok types from npm script * feat: test adding image to the storyblok hero and test using button as link if ctaLink and ctatext returned from Storyblok * chore: add storyblok richtext renderer as dep * feat: add NewsArticle component to render news articles in app - Add News Article component with PageHeader and richtext renderer for content. - Update PageHeader to render formatted date from design Dale H * style: test adding basic layout styles from scss * style: update image css * feat: first pass at blockquote we will use in the richtext examples in news articles * feat: test adding blockquote to the richtext renderer resolver * feat: first pass at NewsLetterSignup using DS components * styles: temp update to blockquote component style * feat: first pass at adding route for news with catch all for news-blogs-and-podcasts (TEMP till slug sturcture agreed) * chore: remove console log * refactor: update NewsArticle component to start using DS components * feat: first pass adding BlogPost component to render blog pages (TODO: move to its own route under pages dir) * refactor: remove empty h2 tag * feat: flesh out news article and move to correct path - Updated the news article to show panels on sidebar - updated StoryblokPageHeader to include tags and date (temporary test until new PageHeader version completed) - Added first pass on StoryblokImage component to use Storyblok image service, adds support for webp and jpeg formats Dale H * feat: first pass on sidebar panels for related news and links - Added first pass StoryblokRelatedLink and StoryblokRelatedNewsLink components - Added Panels to sidebar on NewsArticle Page TODO: update spacing across news article and it's components TODO: add tests for components and pages Dale H * refactor: update related news and link components to use Link component from next-web * chore: remove console.logs * feat: first pass at updating breadcrumbs - Update logic on breadcrumb function - Add path on getBreadcrumbs call, using partly hard coded path. TODO: get path dynamically, within getStaticProps Dale H * refactor: test PageHeader alpha release in StoryblokPageHeader * styles: update featuredImage offset styles to use custom property, so we can update the variable value inside of media queries * feat(ts): udpate type for image and pass rest prop to handle passing of common image attributes * refactor: update newsarticle image to take width and height and maintain 16/9 aspect ratio * test: update image component test and add fallback image for when src is empty * refactor: hardcode Breadcrumbs in news article. Comment out breadcrumb logic from StoryblokPageHeader to test news article * fix: remove updatedBreadcrumb * feat: start richtextRenderer component * styles: remove max width from blockquote to span full width of container * fix: revert StoryblokPageHeader component for stories that use the StoryblokPageHeader blok * feat: first pass on NewsPageHeader to use on news, in-depth, podcast and blog pages * feat: update richtext and news article components - Updated types for richtext component - Add NewsPageHeader and updated Richtext component to NewsArticle page component Dale H * styles: update style to handle images contained within richtext content * feat: first pass at YoutubeEmbed handled by rich text renderer * refactor: update newspageheader to use date util from repo * feat: Update richtext component to render inline images as StoryblokImage component ` * fix: add workaround to handle inline quotes in the richtext field of news stories NOTE: currently the children of the inline quote doesn't render correctly when using the blockquote component in the nodeResolver of storyblok-rich-text-renderer. We bump into a few issues around rendering and correct typing. I've hardcoded the html and imported styles so we can render the correctly styled quote TODO: Update the blockquote component or richtext types to handle passing through richtext quote children to the blockquote component * fix: update richtext interface, revert to initial interface from previous commit * tests: add tests for breadcrumbs and pageheader meta for tag and published date * fix: update breadcrumb name and resources link panel * test: remove test for fetchPriority attribute * refactor: update the inner Link component to use href over to attribute * tests: add relatednewslink tests * tests: first pass adding youtube embed tests TODO: add a field in storyblok for title so we can add an accessible title to the video embed Dale H * tests: firstpass adding tests for StoryblokRichText component * tests: first pass adding tests for Storyblok NewsArticle component * styles: update cta button so inner text wraps * fix: update types for newsarticle, add test to check whether signup action is rendered * tests: update NewsArticle tests * tests: first pass test news article slug page * fix: remove unused imports from richtext component and tests * chore: update design system version for new pageheader * refactor: update the newsarticle component to take new page header over newspageheader. - Update NewsArticle.tsx to use design system PageHeader new version - Add hardcoded breadcrumb values and handle in NewsArticle to be passed to PageHeader component TODO: Possible refactor of NewsPageHeader to be a generic component that can be used on news index and slug pages TODO: Image spacing to be handled via useEffect and ref to image to dynamically update the css varaible for its spacing Dale H ` * chore: rename pageheaderfooter that contains date and tag * refactor: update StoryblokImage and NewsArticle for overlap Added refs to NewsArticle and StoryblokImage so we can target them for updating the custom property value for the image overlap between header and body * style: update spacing around pageheader elements * tests: add further coverage on NewsArticle component * fix: udpate relatedlink and relatednewslink to use resolveStoyblokLink util function * round the pixels down on the featuredImageOffset value * fix: append /m/ for webp source image * fix: add title to Youtube embed for screen reader use * chore: update storyblok types * fix: update youtube iframe embed source value, remove comments * NICE-170 latest from nice & homepage (#175) * NICE-170 Homepage WIP * NICE-170 Latest news WIP * NICE-170 List latest news articles * NICE-170 Featured story WIP * NICE-170 Implement homepage hero * NICE-170 Action banner WIP * NICE-170 Resolved link, promo box WIP * NICE-170 Fix broken links for story objects * NICE-170 Complete "latest from NICE" section * NICE-170 Revert tsconfig, add NewsStory types * NICE-170 Add unit tests for homepage news section & components * NICE-170 Render news grid image as link, fix snapshots * NICE-170 Use better semantics, fix snapshots * NICE-170 misc fixes post review --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nxt 229 updates and fixes (#177) * fix: add empty dependancy array to useEffect to run once on mount, resize event for image * fix: add missing slash on jpeg src * feat: first pass adding author blok * styles: round image for author image * refactor: bring in update to fetchStory function and add author blok * first pass onblog post with listed authors - rough draft * update author component, extract the authorlist from blog post * update interface for blogpost and remove the inlined authorlist * fix: remove trailing comma from tsconfig * refactor: move authorlist to its own component, test author component * add authorList component and extend tests * start moving newspage header and child component into own files * fix: rename blogpost and newsarticle component to keep naming convention for Storyblok blok components * refactor: update AuthorList and tests * tests: start blogpost component tests * styles: update richtext elements vertical spacing and blog post * tests: update blog post slug page tests * tests: update youtube embed tests to include updated title property * tests: add missing slash from returned image source in tests * tests: update snapshot tests related to pageheader updates * styles: remove need for nested grid on BlogPost component, spacing updates on articles and posts * tests: update snapshots related to pageheader updates * tests: add testid to pageheader to test id applied, after pageheader updates * style: add commented styles to take over pageheader secondsection styles in NDS * feat: first pass at StoryblokIframe component * spike: spike nextjs/third-parties and react-lite youtube embeds - Tested next third parties, works but difficulty testing - Tested react-lite-youtube-embed, works and tested. TODO: test accessibility of react-lite-yt-embed further and against standard iframe implementation. TODO: ensure component is typed correctly * chore: remove spiked nextjs/third-parties * fix: remove incomplete import from richtext styles causing no styles to load * feat: use react-lite-youtube-embed for YT embed component * fix: update params and cookie props on on LiteYoutubeEmbed Issue with Safari not autoplaying the video once the poster play button was clicked. Adding cookie=true and updating params to include autoplay=1 solves this issue. * styles: update action banner spacing on newsarticle * styles: first pass normalise spacing to use DS spacing values * styles: normalise blockquote spacing * styles: tidy duplicated spacing * fix: add lodash debounce to handleResize functions on blog and news page components * tests: update breadcrumb test on slug page, and remove expect from youtubeembed test * NXT-250 Refactor fetchstories utility function (#179) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-251 Refactor NewsCard component (#180) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nxt 229 update tests and mocks (#181) * update test utils mocks for blog post and news article * fix: update type of blogpost blok and update tests to use mock util data * fix: udpate type of slug pages for blog and news to pass correct blok types * fix: udpate types and mock for newsarticle component - Update blok prop type and revert back to types generated by Storyblok - remove previous override type for news article - Update StoryblokNewsArticle to use mock data from test utils Dale H * refactor: update authorlist to render author blok over stoyblok component, and add className prop to AuthorBlok * tests: update test to check for author data in blogpost component * tests: update slug page tests on news and blog page slugs, using mock data from utils * tests: add richtext mock data and update tests that use richtext * NICE-169 secondary promo (#182) * NICE-169 Promo sections WIP * NICE-169 Complete Promo Box & tests --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nice 222 news blog style fixes (#184) * styles: update horizontal spacing * styles: update targetting on Youtube embed when child of richtext component * styles: update spacing on featuredImage and page header * refactor: normalise grid layout between newArticle and blogPost, update featuredImage margins * styles: switch featuredImage to 21/9 between sm and md breakpoints * styles: udpate spacing between main content/sidebar, header and action banner * styles: update bottom margin on youtube embed in richtext * styles: add fallback styles for inset and set button hover cursor to pointer * styles: update author styles from crit * styles: remove pageheader bottom margin * NICE-164 Implement Storyblok action banner (#185) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Big snapshot update * NICE-226 draft info page (#190) * NICE-226 Draft info page template * NICE-226 Revert Space ID * NICE-226 Remove log --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Fix linting error * Fix stylelint errors * Style fixes (#191) * fix: update styles to remove BEM convention in modules where we can * fix: add ignore on selector-class-pattern for targeting page header classes from DS in News and Blog components * Stylelint config update * NICE-178 Build the news listing template (#183) * NICE-178 Retrieve list of news articles from storyblok - wip * NICE-178 Add pagination component - wip * NICE-178 Return object with stories and total pages from storyblok response * NICE-178 Add current page and pagination destinations * NICE-178 Calculate total pages for pagination * NICE-178 Redirect to news/articles/ if page is outside bounds * NICE-178 Destructure gss props, return props directly * NICE-178 Componentise news list pagination * NICE-178 Rename prop as they aren't optional * NICE-178 Unit test newsListPagination wrapper * NICE-178 Revert tsconfig to fix issue with module resolution * NICE-178 Unit testing wip. Add npm script so space ID can be set and exported from terminal * NICE-178 Unit tests * NICE-178 Add action banner * NICE-178 Checkout web package.json etc to avoid merge conflict * NICE-178 Add NewsGrid component to articles list layout * NICE-178 Repurpose NewsGrid as NewsList for list pages * NICE-178 Move return in fetchStories to match storyblok-main * NICE-178 Add isListItem variant for news list. Setup basic style and layout for newslist page * NICE-178 Add featured story, sort stories by content.date descending. Update type useage * NICE-178 Add newslist nav component * NICE-178 Add npm script for updating types via shell * NICE-178 Update types * NICE-178 Add SEO * NICE-178 Adjust mobile styles for newscard * NICE-178 Add unit tests. Update snapshots * NICE-178 Add news nav to news index page. Change getActiveLink to match url paths exactly * NICE-178 Fix unit tests * NICE-178 Remove logging line * NICE-178 Move components to News dir to declutter main dir * NICE-178 Update location of pagination component * NICE-178 Prettify * NICE-178 Add unit tests for isNewsListItem variant * NICE-178 Comment out noisy router test * NICE-178 Skip first story on first page as it's featured. Articles index unit test - wip * NICE-178 Remove logger lines * NICE-178 Add todo for gssp on articles index * NICE-178 Unit tests * NICE-178 unit test gssp * NICE-178 remove worthless unit test * NICE-178 Update unit tests. Adjust border thickness for NewsCard * NICE-178 fix nesting level of selectors picked up by linting rule * NICE-178 Move storyblok utils mocks to outer describe * NICE-178 Remove magic number from unit test * NICE-178 Remove unnecessary string conversion in query * NICE-178 Handle storyblok fetchStories error condition * NICE-178 Revised error message * NICE-231 homepage update (#193) * NICE-231 Add homepage spacing, update promo box transparency * NICE-231 Complete Spotlight component * NICE-231 Update spotlight, add new news types * NICE-231 Fix rendering of homepage news links * NICE-231 Update snapshots * NICE-231 Update spotlight component to handle podcasts * NICE-231 Update snapshots, temporarily remove articles listing page unit tests * NICE-231 Fix TS errors * NICE-231 Homepage style updates * NICE-231 Fix snapshots --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Fix linting errors * Fix Card Grid link resolution * Nice 215 blog listing build (#195) * test: udpate snapshot test on newsarticle after updating mock data * udpate news and blog pages to filter scheduled stories according to date field * start validate route/pagination work * NICE-215 validateRouteParams wip * update types around validateRouteParams for storyblok * NICE-215 Handle no stories, parameterise validateRouteParams items and defaults * NICE-215 Fix starts_with after debug * test redirect and update blog page - first pass testing path redirect for out of range pagination pages - update blog listing page with new validateRouteParams - update news article listing page with resolvedUrl Authors: John Davey Dale Hudson * NICE-215 Remove debugging, tidy up out of range function * NICE-215 unit test featured story prop in gssp * NICE-215 pass sbParams differently, clean up types * NICE-215 Handle author content in FeaturedStory component * NICE-215 Change friendlyDate from toLocaleDateString to Intl.DateTimeFormat as this is more performant * NICE-215: update styles and destructure props for author in featuredStory * NICE-215: add first pass at author in newscard * NICE-215: update where we set AuthorHeadingElement * NICE-215: fix no rendering of secondSection if no authors length * NICE-215(fix): layout of author block * update storyblokimage for better handling of serviceOptions * NICE-215: update author card with iscardauthor and headinglevel props - update author to handle different heading levels with default to h4 - add prop to handle author when in card, updated styles for this too * NICE-215: update featured card with updated author component * remove redundant function and logs from SB utils * NICE-215(fix): update how author is handled in blogpost now author component updated * NICE-215(styles): udpate author styles in news and featuredStory cards * NICE-215: update full_slug link urls to absolute urls * NICE-215: udpate perf of background image and author keys - Added Storyblok image service parameters to resize and reduce quality of images to improve image loading times - Changed id for author lists within news and featured story cards * NICE-215: revert jpeg sources ready to refactor to use all image service options * NICE-215: add iframe component to storyblok init * NICE-215(fix): update image to ternary for correct loading of fallback image * NICE-215: fix Youtube embed button aria - remove announce prop from youtube embed, was duplicating aria label values - add fireevent to button click event in YT embed test * NICE-215(refactor): Update StoryblokImage component - Update image component to handle Storyblok image service options - As first pass we only cater for service options width, height, smart, quality - Update components using the StoryblokImage component with updated serviceOptions so we can optimise images currently in use. Dale H * NICE-215: handle error for empty stories array --------- Co-authored-by: John Davey <john.davey@nice.org.uk> Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * NICE-238 Add podcast page (#194) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * return null if author is string (#196) * Nice 215 fix tests (#197) * NICE-215: fix background images using SB image service by encoding brackets () * NICE-215: remove author as string from newsArticle as not part of content model * NICE-215(fix): Update snapshots for link and bgimg - updated snapshots for updated absolute url link handling in news and featured cards - updated snapshots for encoded background image urls containing storyblok image service filters - updated snapshots for featured and news cards with added authors John Davey Dale Hudson * fix pagination tests with updated prop names * NICE-215(fix): update snapshots from image component updates * NICE-215(fix): udpate tests after AuthorList refactor * NICE-215(fix): handle blogpost authors when array empty Handle blog posts author when they are pubilshed with an unpublished or no author attached. John D Dale H * Nice 231 homepage update (#198) * NICE-231 Add homepage spacing, update promo box transparency * NICE-231 Complete Spotlight component * NICE-231 Update spotlight, add new news types * NICE-231 Fix rendering of homepage news links * NICE-231 Update snapshots * NICE-231 Update spotlight component to handle podcasts * NICE-231 Update snapshots, temporarily remove articles listing page unit tests * NICE-231 Fix TS errors * NICE-231 Homepage style updates * NICE-231 Fix snapshots * NICE-231 Resolve merge conflicts * NICE-231 Homepage UI updates * NCIE-231 Limit news card to 1 author * NICE-231 Update author UI, change absolute URLs * NICE-231 Fix snapshots * NICE-231 Update Promobox mock data and snapshot --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nice 217 podcast listing page (#199) * NICE-217: podcast listing page, initial setup - Condtionally render newscard image - Update newscard author, return null if author is string D Hudson J Davey * NICE-217 Update podcast listing page - Make newsCard image optional - Fix broken unit test by removing redundant console log - Early return if author is just a string - Make newsListNav propless with destinations contained as they won't change - First pass at podcast listing page Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NICE-217 Add unit tests to cover optional showImage Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NICE-217 Update lead text on podcast list page * NICE-217: update breadcrumbs with absolute links --------- Co-authored-by: John Davey <john.davey@nice.org.uk> Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NICE-248 Resolve author linting issues (#200) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NICE-218 First pass at in-depth listing page (#201) Co-authored-by: John Davey <john.davey@nice.org.uk> Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * Nxt 270 news homepage (#202) * NXT-270 News homepage WIP * NXT-270 More news homepage WIP * NXT-270 Complete news homepage --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Fix linting issue * Add remote pattern image config * NICE 253 Refactor init storyblok (#204) * proof that separating initStoryblok enables proper mocking * streamline component imports for storyblokcomponent pages Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> --------- Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NICE-253 Fix issue with initStoryblok refactor breaking the catch all slug. Optimise component object for lean bundle. (#207) Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NXT-269(fix): filter articles out of latest news with future date (#208) * NICE-254 Handle image optimisation (#205) * NICE-254 Optimise images coming from storyblok image service. Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NICE-254 Update snapshots * NICE-254 Add optimise image util and unit test * NICE-254 Reinstate storyblokhero --------- Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * Nxt 292 blog page fixes from feedback (#209) * NXT-292: fix blog post author and author list headings * NXT-292: fix headings, not to render when no content * Merge storyblok-main Co-authored-by: John Davey <johndavey72@users.noreply.github.com> --------- Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * Nxt 292 blog page fixes from feedback (#210) * NXT-292: fix blog post author and author list headings * NXT-292: fix headings, not to render when no content * Merge storyblok-main Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * NXT-292 linting * NXT-292 merge sbmain --------- Co-authored-by: John Davey <johndavey72@users.noreply.github.com> Co-authored-by: John Davey <john.davey@nice.org.uk> * NXT-292: update homepage snapshot * NICE-255 Change enablejsapi from 0 to 1 (#212) * NICE-250 Managing focus on pagination route changes (#214) * spike pagination announcement for screenreader users - Add a div calling out the current page and total pages when navigating via screenreader - Update focus to hidden div, just above NewsList - this is one option of a few we need to test, preferably with screenreader users. - Options to go through linked on ticket: NICE-250 * NICE-250 Use Announcer component for pagination changes * NICE-250 add aria-hidden on focus element to prevent screen reader reading out 'blank' * NICE-250 Add announcer to other list pages. Pagination focus hook wip Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NICE-250 Roll out pagination focus to other list pages. Add component for pagination focus. * NICE-250 Remove console log * NICE-250 Add unit tests for pagination focus component * NICE-250 Refactor announcer code into a re-useable component * NICE-250 Remove unused deps and commented out code --------- Co-authored-by: dalecreativ <dalefrontend@gmail.com> Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NICE-260 Calculate canonical urls for slug pages and unit test (#216) Co-authored by Dale Hudson * Nxt 283 font size fixes (#215) * NXT-283: Update root font size to handle narrow viewport sizes. Co-authored by: John Davey * NXT-283: update font sizes for h1 on smaller screen sizes Targetting hero and page headings for home, news article, and blog post pages. Co-authored by: John Davey * NXT-283 Adjust h1 font-size for narrow viewport. Apply word-break and hyphenation for very narrow viewports Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> --------- Co-authored-by: John Davey <john.davey@nice.org.uk> Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * Nice 261 refactor image optimisation (#217) * NXT-261: refactor constructStoryBlokImageSrc to encodeParens and handle background images Co-authored by: John Davey. * NXT-261: null check image on podcast slig page * NXT-261: remove optimiseImage import from tests * NICE-261: update failing snapshot from encodeParens update (#218) * Nice 266 homepage spacing (#221) * NICE-266 Update homepage spacing; restore blog authors on news homepage * NICE-266 Miscellaneous QA fixes * NICE-266 Update blog page title * NICE-266 Set podcast image to unoptimised * NICE-266 Prevent future articles from appearing on news homepage * NICE-266 Pedantry --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nice 266 homepage spacing (#223) * NICE-266 Update homepage spacing; restore blog authors on news homepage * NICE-266 Miscellaneous QA fixes * NICE-266 Update blog page title * NICE-266 Set podcast image to unoptimised * NICE-266 Prevent future articles from appearing on news homepage * NICE-266 Pedantry * NICE-266 Set base URL for podcast image --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nice 266 homepage spacing (#224) * NICE-266 Update homepage spacing; restore blog authors on news homepage * NICE-266 Miscellaneous QA fixes * NICE-266 Update blog page title * NICE-266 Set podcast image to unoptimised * NICE-266 Prevent future articles from appearing on news homepage * NICE-266 Pedantry * NICE-266 Set base URL for podcast image * NICE-266 Pluralise podcasts label --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NICE-283 spacing again (#229) * NICE-283 Make spacing and content changes * NICE-283 Spacing, routing, text updates * NICE-283 A11y fixes * NICE-238 Amend news article action banner spacing * NICE-283 Update action banner spacing * NICE-283 Restore access token config --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Update snapshots * NXT-276 Update default image with base URL from config * NXT-217 Use Ocelot for Storyblok cacheing (#225) * NXT-217 Installed Ocelot.Administration NuGet package * NXT-217 Added Storyblok config to Ocelot routes * NXT-217 Configure Storyblok to use Ocelot * NXT-217 Add Ocelot Administration endpoint * NXT-217 Refactor Ocelot cache config settings * NXT-217 Add Ocelot for Storyblok section to readme * NXT-217 Update Readme --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NICE-291 Return 404 for root catch all unless env var enables it (#230) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-346 Revert env var config, add test page (#232) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Nxt 346 revert env vars (#233) * NXT-346 Revert env var config, add test page * NXT-346 Add test env var to config --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * fix: update default ocelotEndpoint to empty value * Remove env var from next config * NXT-319, NICE-251 unit testing utility functions and news index pages, adding response status from SB API (#231) * initial commit * add boilerplate for get server side props * update validaterouteparams to return redirect if page is less than 1 TODO: alternative approach of assigning page to 1 if query.page is less than 1 * update mock data and refactor mocks for listing page * stub out getStoryblokAPi get method and modify response per test * Added json files for various responses Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * Added further test for fetchStory util co-authored by Dale Hudson * NXT-319 Add server error unit test Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NXT-319 Add unit tests for fetchStories co-authored by Dale Hudson * NXT-319 Add fetchLinks unit tests Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NXT-319 Add fetchLinks unit tests Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NXT-319 Remove promise.reject from fetchStories to prevent unreachable code issue. Tidy up imports in storyblok utils. Co-authored by Dale Hudson * NXT-319 Change mock links success response to match item returned from storyblok/react. Add note to investigate jest setup mocking issue * NXT-319 Add getBreadcrumbs unit tests * NXT-319 Add error handling to friendlyDate. Unit test friendlyDate * NXT-301: add test coverage for getNewsTypes * NXT-319: update validateRouteParams query.page to be set to one if less than 1 * NXT-319: start validateRouteParams test coverage * NXT-319: update news index page tests after updates * NXT-319: test for success and error responses from fetchStories returned to valiudateRouteParams * NXt-319: remove skip test validateRouteParam * NXT-319 Handle no stories on index pages. Return not founds for incorrect query page param. Refactor and add unit tests for validateRouteParams. Remove log lines. Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NXT-319 Remove unused spy. Add unit test todos for news articles listing" * NXT-319 Add unit test to check undefined return from getSlugFromParams. Remove unused dep * NXT-319 Add unit tests for news article list page. Refactor around thrown errors from validateRouteParams. Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NXT-319 Unit tests of news articles renderins. Gssp correct props unit test. * NXT-319 Rendering unit tests for news articles listing Co-authored by: Dale Hudson * NXT-319 Convert in-depth gssp to try catch format. Adjust types to separate error props from success props. * NXT-319 Move isError util function to storyblok utils and unit test. Make public facing error messages more generic". Update snapshot. * NXT-319 Update blog posts index gssp to try catch format. Separate error and success props and types" * NXT-319: update podcast gssp to handle errors from api * NXT-319 Remove resolved url from gssp args. Update unit tests to reflect removal of API details in error messages. * NXT-319 Add unit tests for news index pages * NXT-319 Add further unit tests for news index pages. * NXT-319 Refactor blog slug page. Added blog slug page unit tests. Added mock success response for blog post. * NXT-319 Refine types for fetchStory, blog slug page and unit test mock data types. Remove error property from fetchStories. Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> * NXT-319 Remove unused deps * NXT-319: remove test backup files * NXT-319: refactor news article slug with TODO comments for discussion * NXT-319: add test coverage for articles slug page * NXT-319: add TODO for discussion on fetchStory * NXT-319: add snapshot for news article page * NXT-319: remove redundant imports from news slug tests * NXT-319: add mockData for podcast page * NXT-319: refactor podcast slug page in line with other slug pages * NXT-319: add tests for podcast slug page with snapshot * NXT-319: udpate type for newsarticle fetch and props. Add logger on getadditionalMetaTag failure * NXT-319: add useMemo back to getAdditionalMetatags in blog post page * NXT-319: add logger to getAdditionalMetaTags in podcast page * NXT-319: remove unused imports from news slug page * Add further test coverage for podcast slug page - updated snapshots Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * NXT-319: remove redundant mocks from blog slug page tests * NXT-319 Error handling for news index and unit test - wip * NXT-319: test news homepage, gssp and error handling * NXT-319: add tests for homepage and handle fetch errors from API. Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * NXT-319: fix linting errors Co-authored by: John Davey * NXT-319: update error handling in utils. consistant messaging and var naming * NXT-319: remove TODOs * NXT-319: update homepage snapshot and test with error message update Co-authored by: John Davey * NXT-319: replace isError from utils with lodash isError equivalent Co-authored by: John Davey * NXT-319: remove isError import from test * NXT-319: move TODOs to tech debt and remove redundant comments. - update podcast listing mock data Co-authored by: John Davey --------- Co-authored-by: John Davey <john.davey@nice.org.uk> Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * Nxt 301 blog page fixes from feedback (#219) * NXT-301: order list of authors alphabetically * NXT-301: add border and spacing to author section in blog header * NXT-301: update author section spacing and remove bottom margin from any richtext component last child element * NXT-301: remove TODOs that have been resolved through updates or discussions * NXT-301: increase on author name if img is included * NXT-301: add role presentation to images carrying no alt text * NXT-301: update width of PageHeader fullwidth-container Full width container updates changed the alignment of pageheader to the rest of content on blog and news articles. Updated so content between header and main content is aligned. * NXT-301: add stylelint ignore rule for global styles targetting page-header * NXT-301: update author blok border with token for colour * NXT-301: revert page header container horizontal spacing. * NXT-301: pageheader author block spacing - Only onclude border and bottom padding over breakpoint * NXT-301: first pass at updating the richtext image aspect ratios based on storyblok image width and heights * NXT-301: update test data to have accurate url structure for SB CDN images * NXT-301: update displayed dates to use the time tag. - update components that display dates to use time tage - update the mock data for dates - refactor the news/blog header meta to use getNewsType util TODO: refactor tag and date into meta component for reuse across other components Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * NXT-301: update richtext image - update aspect ratio, width and height attrs using SB image url path to get width and height * NXT-301: replace storyblok cdn and link domain from url mock data. Mitigate calls to actual url. * NXT-301: upate storyblok-data mock urls * NXT-301: update snapshots after mock data changes to urls * NXT-301: update styles to nudge featured image - Nudge pixels on page header and featuredimage across news and blog page templates - Keeps margin top within 2rem of page header bottom content TODO: revisit handling of featuredImage offsets. Adding to tech debt for post go live Dale H * NXT-301: remove TODOs and put in tech debt log for discussion * NXT-301: move setComponent causing rendering issues --------- Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * fix: add back lint disable line for global targetting of css from DS * Main merge (#242) * NXT-246 Indicator Old URL mixed casing - redirects not working correctly (#192) * NXT-246 Added middleware to convert all resolved urls to lowercase Next js is routing to different pages/components if the url uses mixed casing compared to lowercase. This fixes that. * NXT-246 Fixed check to compare mapped indicator url with resolved url * NXT-246 Fixed linting issues * NXT-246 Added unit test for middleware function * NXT-330 Indicators change chapters to chapter (#226) * change chapters to chapter in indicators [slug] and update unit tests * NXT-330 Removed obsolete snapshots --------- Co-authored-by: John Davey <john.davey@nice.org.uk> * NXT-337 Fix for empty product panel (#227) * NXT-337 Fix for missing _embedded.niceIndevPanel Undefined when no documents are present. * NXT-337 Further null checks for panel list * NXT-343 Exclude next-js manifest files from url lowercasing middlewar… (#228) * NXT-343 Exclude next-js manifest files from url lowercasing middleware and unit test. Co-authored-by: John Holland * NXT-343 Modify comment to trigger TC build * NXT-353 Remove middleware page (#235) * NXT-353 Remove middleware page * NXT-353 Reinstate error causing middleware files * NXT-242 Add Terraform config (#176) * NXT-210 Add test logs to status page * NXT-210 Move logging test into forms dir * NXT-242 Add Terraform config * NXT-242 Update vars * NXt-242 Change ip to non public * NXT-241-new k6 scripts - js files and json files * NXT-242 Change bucket name * NXT-242 Add values into json * NXT-242 Update tfvars * NXT-242 Change to octo vars * NXT-242 Update tf vars * NXT-242 Fix typo * NXT-242 Fix issues * NXT-242 Update tfvars * NXT-241-updates to perfomance test k6 scripts * NXT-241-revert changes to vscode settings * NXT-241-remove typeahead terms from combined scripts and cleanup * NXT-241-added more paths to the combined json file * NXT-241-edited galp_pages test data * NXT-242 Added load balancer config to tf * NXT-242 Add build number to config dir * NXT-242 Added diagnostic info * NXT-242 Update efs volume path * NXT-242 Update var names * NXT-242 Add node env * NXT-242 Refactor TF * NXT-242 Revert config.yml * NXT-242 Update default.yml * NXT-242 Remove comment * NXT-242 Variablise CPU and Memory * NXT-242 Remove breaking characters * NXT-242 Add variable characters * NXT-242 Change variable type to number * NXT-242 Move pino-mqrc.json * NXT-242 Fix pino-mq config problem * NXT-242 Fix issues with pino-mq * NXT-288 Enable AWS Exec # Conflicts: # aws/terraform/terraform.tfvars # aws/terraform/variables.tf * NXT-242 Add server count variable, remove cloudwatch variable * NXT-241 Add basic test and change dir * NXT-241-fix-merge-commits * NXT-241-add-more-test-data * NXT-241-add-stages-plus-the-noConnectionReuse * NXT-241-prettier-fixes * NXT-242 Add teamcity aws config * NXT-242 Add extra stuff to k6 testing file * NXT-347 Update testing files * NXT-242 Add gitignore * NXT-242 Add readme.md * NXT-350 Add curl to Docker * NXT-350 Add healthcheck * NXT-242 Remove PM2 * NXT-242 Update readme * NXT-210 Update logtest.page.tsx * NXT-242 Remove performance testing * NXT-242 Apply TF format * NXT-242 Fix typo * NXT-242 Add comments to readme * NXT-242 Update readme --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: FunmiOsho <60607637+FunmiOsho@users.noreply.github.com> Co-authored-by: lizaadebowale <liza.adebowale@nice.org.uk> Co-authored-by: L-Vazquez <37586672+L-Vazquez@users.noreply.github.com> * Update package lock --------- Co-authored-by: John Holland <john.holland@nice.org.uk> Co-authored-by: John Davey <john.davey@nice.org.uk> Co-authored-by: John Davey <66066685+johndavey72@users.noreply.github.com> Co-authored-by: Colin M <12713935+colinmazu@users.noreply.github.com> Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: FunmiOsho <60607637+FunmiOsho@users.noreply.github.com> Co-authored-by: lizaadebowale <liza.adebowale@nice.org.uk> Co-authored-by: L-Vazquez <37586672+L-Vazquez@users.noreply.github.com> * Nice 249 sitemap generation (#245) * NICE-249 Fetch news content for sitemap from Links API, unignore sitemaps from Git, edit static sitemap from Orchard * NICE-249 Complete sitemap generation * NICE-249 Scrub auto-generated sitemap --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-345 Consolidate preview and public access tokens, remove env test page (#246) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-369 Add redirects for news section (#247) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Remove cv parameter from sbParams if Ocelot not present * Fix incorrect news redirect * Fix failing test due to CV parameter * Storyblok merge main (#250) * NXT-246 Indicator Old URL mixed casing - redirects not working correctly (#192) * NXT-246 Added middleware to convert all resolved urls to lowercase Next js is routing to different pages/components if the url uses mixed casing compared to lowercase. This fixes that. * NXT-246 Fixed check to compare mapped indicator url with resolved url * NXT-246 Fixed linting issues * NXT-246 Added unit test for middleware function * NXT-330 Indicators change chapters to chapter (#226) * change chapters to chapter in indicators [slug] and update unit tests * NXT-330 Removed obsolete snapshots --------- Co-authored-by: John Davey <john.davey@nice.org.uk> * NXT-337 Fix for empty product panel (#227) * NXT-337 Fix for missing _embedded.niceIndevPanel Undefined when no documents are present. * NXT-337 Further null checks for panel list * NXT-343 Exclude next-js manifest files from url lowercasing middlewar… (#228) * NXT-343 Exclude next-js manifest files from url lowercasing middleware and unit test. Co-authored-by: John Holland * NXT-343 Modify comment to trigger TC build * NXT-353 Remove middleware page (#235) * NXT-353 Remove middleware page * NXT-353 Reinstate error causing middleware files * NXT-242 Add Terraform config (#176) * NXT-210 Add test logs to status page * NXT-210 Move logging test into forms dir * NXT-242 Add Terraform config * NXT-242 Update vars * NXt-242 Change ip to non public * NXT-241-new k6 scripts - js files and json files * NXT-242 Change bucket name * NXT-242 Add values into json * NXT-242 Update tfvars * NXT-242 Change to octo vars * NXT-242 Update tf vars * NXT-242 Fix typo * NXT-242 Fix issues * NXT-242 Update tfvars * NXT-241-updates to perfomance test k6 scripts * NXT-241-revert changes to vscode settings * NXT-241-remove typeahead terms from combined scripts and cleanup * NXT-241-added more paths to the combined json file * NXT-241-edited galp_pages test data * NXT-242 Added load balancer config to tf * NXT-242 Add build number to config dir * NXT-242 Added diagnostic info * NXT-242 Update efs volume path * NXT-242 Update var names * NXT-242 Add node env * NXT-242 Refactor TF * NXT-242 Revert config.yml * NXT-242 Update default.yml * NXT-242 Remove comment * NXT-242 Variablise CPU and Memory * NXT-242 Remove breaking characters * NXT-242 Add variable characters * NXT-242 Change variable type to number * NXT-242 Move pino-mqrc.json * NXT-242 Fix pino-mq config problem * NXT-242 Fix issues with pino-mq * NXT-288 Enable AWS Exec # Conflicts: # aws/terraform/terraform.tfvars # aws/terraform/variables.tf * NXT-242 Add server count variable, remove cloudwatch variable * NXT-241 Add basic test and change dir * NXT-241-fix-merge-commits * NXT-241-add-more-test-data * NXT-241-add-stages-plus-the-noConnectionReuse * NXT-241-prettier-fixes * NXT-242 Add teamcity aws config * NXT-242 Add extra stuff to k6 testing file * NXT-347 Update testing files * NXT-242 Add gitignore * NXT-242 Add readme.md * NXT-350 Add curl to Docker * NXT-350 Add healthcheck * NXT-242 Remove PM2 * NXT-242 Update readme * NXT-210 Update logtest.page.tsx * NXT-242 Remove performance testing * NXT-242 Apply TF format * NXT-242 Fix typo * NXT-242 Add comments to readme * NXT-242 Update readme --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: FunmiOsho <60607637+FunmiOsho@users.noreply.github.com> Co-authored-by: lizaadebowale <liza.adebowale@nice.org.uk> Co-authored-by: L-Vazquez <37586672+L-Vazquez@users.noreply.github.com> * NXT-339 Update readme (#240) * NXT-353,NXT-343 404 errors on nextweb (#234) * NXT-343 Exclude next-js manifest files from url lowercasing middleware and unit test. Co-authored-by: John Holland * NXT-343 Modify comment to trigger TC build * NXT-343 Change url.pathname check to include _next/data and _next/static paths to avoid request errors * NXT-343 Add test for _next/data url condition * NXT-343 lowercase any mixed case indicators or guidance urls * NXT-343 Remove middleware page and unit tests * NXT-343 Hardcode indicators legacy redirects * NXT-343 Move indicators redirects from file directly to the config * NXT-343 Trying separate redirects file again * Revert "NXT-343 Trying separate redirects file again" This reverts commit f8d73685610fe09b773a9dd76bf3b26be8b8df01. * NXT-343 Correct redirects where there are duplicates with same case or missing lowercase version * Update package lock --------- Co-authored-by: John Holland <john.holland@nice.org.uk> Co-authored-by: John Davey <john.davey@nice.org.uk> Co-authored-by: John Davey <66066685+johndavey72@users.noreply.github.com> Co-authored-by: Colin M <12713935+colinmazu@users.noreply.github.com> Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: FunmiOsho <60607637+FunmiOsho@users.noreply.github.com> Co-authored-by: lizaadebowale <liza.adebowale@nice.org.uk> Co-authored-by: L-Vazquez <37586672+L-Vazquez@users.noreply.github.com> * Storyblok main merge conflicts (#252) * NXT-246 Indicator Old URL mixed casing - redirects not working correctly (#192) * NXT-246 Added middleware to convert all resolved urls to lowercase Next js is routing to different pages/components if the url uses mixed casing compared to lowercase. This fixes that. * NXT-246 Fixed check to compare mapped indicator url with resolved url * NXT-246 Fixed linting issues * NXT-246 Added unit test for middleware function * NXT-330 Indicators change chapters to chapter (#226) * change chapters to chapter in indicators [slug] and update unit tests * NXT-330 Removed obsolete snapshots --------- Co-authored-by: John Davey <john.davey@nice.org.uk> * NXT-337 Fix for empty product panel (#227) * NXT-337 Fix for missing _embedded.niceIndevPanel Undefined when no documents are present. * NXT-337 Further null checks for panel list * NXT-343 Exclude next-js manifest files from url lowercasing middlewar… (#228) * NXT-343 Exclude next-js manifest files from url lowercasing middleware and unit test. Co-authored-by: John Holland * NXT-343 Modify comment to trigger TC build * NXT-353 Remove middleware page (#235) * NXT-353 Remove middleware page * NXT-353 Reinstate error causing middleware files * NXT-242 Add Terraform config (#176) * NXT-210 Add test logs to status page * NXT-210 Move logging test into forms dir * NXT-242 Add Terraform config * NXT-242 Update vars * NXt-242 Change ip to non public * NXT-241-new k6 scripts - js files and json files * NXT-242 Change bucket name * NXT-242 Add values into json * NXT-242 Update tfvars * NXT-242 Change to octo vars * NXT-242 Update tf vars * NXT-242 Fix typo * NXT-242 Fix issues * NXT-242 Update tfvars * NXT-241-updates to perfomance test k6 scripts * NXT-241-revert changes to vscode settings * NXT-241-remove typeahead terms from combined scripts and cleanup * NXT-241-added more paths to the combined json file * NXT-241-edited galp_pages test data * NXT-242 Added load balancer config to tf * NXT-242 Add build number to config dir * NXT-242 Added diagnostic info * NXT-242 Update efs volume path * NXT-242 Update var names * NXT-242 Add node env * NXT-242 Refactor TF * NXT-242 Revert config.yml * NXT-242 Update default.yml * NXT-242 Remove comment * NXT-242 Variablise CPU and Memory * NXT-242 Remove breaking characters * NXT-242 Add variable characters * NXT-242 Change variable type to number * NXT-242 Move pino-mqrc.json * NXT-242 Fix pino-mq config problem * NXT-242 Fix issues with pino-mq * NXT-288 Enable AWS Exec # Conflicts: # aws/terraform/terraform.tfvars # aws/terraform/variables.tf * NXT-242 Add server count variable, remove cloudwatch variable * NXT-241 Add basic test and change dir * NXT-241-fix-merge-commits * NXT-241-add-more-test-data * NXT-241-add-stages-plus-the-noConnectionReuse * NXT-241-prettier-fixes * NXT-242 Add teamcity aws config * NXT-242 Add extra stuff to k6 testing file * NXT-347 Update testing files * NXT-242 Add gitignore * NXT-242 Add readme.md * NXT-350 Add curl to Docker * NXT-350 Add healthcheck * NXT-242 Remove PM2 * NXT-242 Update readme * NXT-210 Update logtest.page.tsx * NXT-242 Remove performance testing * NXT-242 Apply TF format * NXT-242 Fix typo * NXT-242 Add comments to readme * NXT-242 Update readme --------- Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: FunmiOsho <60607637+FunmiOsho@users.noreply.github.com> Co-authored-by: lizaadebowale <liza.adebowale@nice.org.uk> Co-authored-by: L-Vazquez <37586672+L-Vazquez@users.noreply.github.com> * NXT-339 Update readme (#240) * NXT-353,NXT-343 404 errors on nextweb (#234) * NXT-343 Exclude next-js manifest files from url lowercasing middleware and unit test. Co-authored-by: John Holland * NXT-343 Modify comment to trigger TC build * NXT-343 Change url.pathname check to include _next/data and _next/static paths to avoid request errors * NXT-343 Add test for _next/data url condition * NXT-343 lowercase any mixed case indicators or guidance urls * NXT-343 Remove middleware page and unit tests * NXT-343 Hardcode indicators legacy redirects * NXT-343 Move indicators redirects from file directly to the config * NXT-343 Trying separate redirects file again * Revert "NXT-343 Trying separate redirects file again" This reverts commit f8d73685610fe09b773a9dd76bf3b26be8b8df01. * NXT-343 Correct redirects where there are duplicates with same case or missing lowercase version * Update package lock --------- Co-authored-by: John Holland <john.holland@nice.org.uk> Co-authored-by: John Davey <john.davey@nice.org.uk> Co-authored-by: John Davey <66066685+johndavey72@users.noreply.github.com> Co-authored-by: Colin M <12713935+colinmazu@users.noreply.github.com> Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: FunmiOsho <60607637+FunmiOsho@users.noreply.github.com> Co-authored-by: lizaadebowale <liza.adebowale@nice.org.uk> Co-authored-by: L-Vazquez <37586672+L-Vazquez@users.noreply.github.com> * NXT-345 Storyblok api key update (#239) * NXT-345: update initStoryblok to use new publicRunTime variable for SB. - add new yaml config variable - refactor catch all slug to logic in other pages Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * NXT-345: add test coverage for initStoryblok Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * NXT-345: add generic error as global const - update generic error message to global const - update tests to cover error handling Co-authored-by: John Davey <johndavey72@users.noreply.github.com> * NXT-345: remove unused import * NXT-345 Linting fixes * NXT-345 update generic error message in SB util tests * NXT-345: remove server config properties and mocks --------- Co-authored-by: John Davey <johndavey72@users.noreply.github.com> Co-authored-by: John Davey <john.davey@nice.org.uk> * Regenerated package lock (#257) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * Fix snapshot (#258) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-370 Add datetime to nextweb config dir name * NXT-374 remove middleware (#260) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-380 Update newsletter signup URL and related snapshots/references (#262) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-381 Rename chapters to chapter in indicator slug subdirectories for evidence and resources (#263) * NXT-382 Move hard-coded auth setting into config vars (#264) Co-authored-by: Chris Barker <chris.barker@nice.org.uk> * NXT-217 Add Variable substitution to Global Configuration * NXT-217 Add Debug console log * NXT-217 Flush Storyblok cache if using Ocelot * NXT-217 Convert TtlSeconds to octo variable * NXT-217 Remove quotes from TtlSeconds variable * NXT-217 Removed merge errors * NXT-217 - fix author images not displaying with ocelotEndpoint enabled - updated storyblok/react library resolves caching issues around resolve_relations and blog authors. - types for SB stories required updating to ISbStoryData after storyblok/react bump to latest. * NXT-217 chore - update storyblok types from Storyblok space updates. * NXT-217 Remove admin module * NXT-217 checkout updated files from main to avoid merge conflicts --------- Co-authored-by: Chris Barker <chris@christopherbarker.co.uk> Co-authored-by: Chris Barker <chris.barker@nice.org.uk> Co-authored-by: John Holland <john.holland@nice.org.uk> Co-authored-by: Claire Hunter <claire.hunter@nice.org.uk> Co-authored-by: John Davey <66066685+johndavey72@users.noreply.github.com> Co-authored-by: Dale Hudson <dalefrontend@gmail.com> Co-authored-by: John Davey <john.davey@nice.org.uk> Co-authored-by: John Davey <johndavey72@users.noreply.github.com> Co-authored-by: Dale Hudson <dalecreativ@users.noreply.github.com> Co-authored-by: Dale H <dale.hudson@nice.org.uk> Co-authored-by: Sue Garner <33061045+suegarner@users.noreply.github.com> Co-authored-by: FunmiOsho <60607637+FunmiOsho@users.noreply.github.com> Co-authored-by: lizaadebowale <liza.adebowale@nice.org.uk> Co-authored-by: L-Vazquez <37586672+L-Vazquez@users.noreply.github.com> Co-authored-by: Sue Garner <Sue.Garner@nice.org.uk>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.