Merge Dev into Master #5478
Merged
Merge Dev into Master #5478
Conversation
This file contains 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
…ns (#5446) * EES-5632 [WIP] Refactoring BE to return release redirects grouped by publication * EES-5632 [WIP] Integration tests * EES-5632 Enabling a different Azurite TestContainer instance to be used across each test so that the stored data doesn't interfere with other tests * EES-5632 FE - Changing the view-model field names to match the new BE field names * EES-5632 Some requested changes as per PR review * EES-5632 removing some duplicated tests * EES-5632 Reformatting tests * EES-5632 remove unused import
EES-5732 prevent ui test failures after data replacement
EES-5543: Correct BoundaryLevel property in ChartDataSetConfig.
EES-5735: Add frontend error mapping for API data set replacement.
… instead optional
EES-5543 bugfix: dataSetConfig BoundaryLevels, no longer defaulted to 0
…ed to MethodologyVersion, Publication and Release.
EES-5632 Updating `ContentDbContextModelSnapshot`
…using an old Release slug in the URL (#5419) * EES-5633 Refactoring the redirect logic to capture release redirects, and to allow us to define more accurately which routes we expect redirects for * EES-5633 Fixing URLPattern Regex + Adding Jest tests + Fixing bug where we weren't isolating a module between test runs The module state (cache) of the `redirectPages` module was interfering with subsequent runs of the tests. So we have used Jest's built-in `isolateModulesAsync` method to ensure that the module state is not carried over between test runs. * EES-5633 Adding tests for child-routes that aren't accounted for in the redirect patterns + organising tests into groups * EES-5633 Removing comments * EES-5633 Refactoring to allow releases to have the same slugs across different publications * EES-5633 Simplifying business logic * EES-5633 Fixing FE test * EES-5633 Removing invalid fast-track and permalink routes * EES-5633 Removing the `/data-tables/{publication-slug}` path from the list of matched redirect URL Patterns * EES-5633 Adding the `/data-tables/{publication-slug}` path back into the list of matched redirect URL Patterns * EES-5633 Removing Lodash usage in middleware to fix build error Was previously getting this error when running `pnpm run build` in the frontend project: ``` Failed to compile. ./..\..\node_modules\.pnpm\[email protected]\node_modules\lodash\lodash.js Dynamic Code Evaluation (e. g. 'eval', 'new Function', 'WebAssembly.compile') not allowed in Edge Runtime Learn More: https://nextjs.org/docs/messages/edge-dynamic-code-evaluation Import trace for requested module: ./..\..\node_modules\.pnpm\[email protected]\node_modules\lodash\lodash.js ./src\middleware\pages\redirectPages.ts ./src\middleware.ts ``` * EES-5633 Removing unnecessary null check
…ed the data block size report until we need to use it.
…_DataBlockQuerySizeController EES-4336 Remove `DataBlockQuerySizeController`
…st-snapshots chore(tests): Update test snapshots 18-12-2024
…update naming convention for "DataSet" in validation messaging.
EES-5047: Implement character limits
EES-5587 refactor reordering filters and indicators
…st-snapshots chore(tests): Update test snapshots 19-12-2024
N-moh
requested review from
ntsim,
duncan-at-hiveit,
bennettstuart,
tomjonesdev and
amyb-hiveit
amyb-hiveit
approved these changes
Dec 19, 2024
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.
This PR merges Dev branch into Master for upcoming deploys to Pre-Production and Production.