Dev bootstrap current#617
Closed
droberts-ctrlo wants to merge 25 commits into
Closed
Conversation
* Fix for regressive error in display conditions * Modify code to no longer remove erroneous conditions and instead create error * Remove change to typing
* Completed error logging to Audit Also add logging upload to logging JS * Update to use `Log::Report` * Change code as per review Also remove extrenous logging and debugging code that was added during development of this feature. * Fix erroring test * Return JSON as required
Fix record download when approval records exist (D:1350)
…es-fix Stop automatic replies to emails (D1065)
…ng-patch Added appropriate error for blank field name heading in record imports (D1099)
* Update cache to ignore values over 250 characters * Fix cache code - Fix incorrect return on calc cache code - Add unit test to ensure correct write operation * Fix test for caching and added search test * Update test to check for value removal * Added comment to test and removed debug code
Remove logging line which was causing an error - HOTFIX - D1511
* Formatted test and config code and updated test files Also removed extra yarn commands that weren't needed, and automated download of browserslist updates on build. * Update autosave.test.ts * Updated filter where typeahead is missed * Update component.js * Reset erroneous changes * Removed invalid import where required * Update package.json * Updated error handler Updated error handler * Updated ts-ignore where this was causing issues with linting and checking correctness * Updated layoutbuilder to ensure cypress tests pass
* Formatted test and config code and updated test files Also removed extra yarn commands that weren't needed, and automated download of browserslist updates on build. * Updated filter where typeahead is missed * Update component.js * Reset erroneous changes * Updated error handler * Formatted test and config code and updated test files Also removed extra yarn commands that weren't needed, and automated download of browserslist updates on build. * Added editorconfig and upgraded ESLint to current version * Formatted and changed code with ESLint fixes - functionality has not changed Removed cjs from eslint - cjs are used for internal development files, and should never be included * Fixed formatting and code where this was broken on merge with current dev branch Updated files where dev changes weren't included
Contributor
Author
|
Cypress failure is a transient issue - in local testing this does not occur - I cannot re-run the test to check |
Also update VERSION to 2.8.2 and GADS.pm to reflect the new version number.
Contributor
Author
|
No longer required |
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
4 participants
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.
Rebase into current dev branch