-
Notifications
You must be signed in to change notification settings - Fork 9
Conversation
Updated error handler
...changed Removed cjs from eslint - cjs are used for internal development files, and should never be included
... dev branch Updated files where dev changes weren't included
Updated documentation Fixed formatting and code where this was broken on merge with current dev branch Added fix for error on documentComponent Updated code files with changes as required where they were included in dev Updated file where missing function wasn't included
Further package changes and TSConfig updates Removed knip
Updated code files with changes as required where they were included in dev
Updated to use new switch component for fullscreen
Updated formatting as this didn't update on prev commit properly
Fixed cypress tests
Fixed cypress tests
Updated as per testing results Further changes due to buttons not displaying correctly Updated package file
Removed bootstrap select Bootstrap select is no longer compatible with Bootstrap v5 - the version that is compatible is still in beta and contains bugs that make it unsuitable for our use-case Updated code to fix WCAG issue Updated changes as per ticket Updated datatables for more consistent colours Fix for fullscreen switch not working as expected Modified header for correct placement of pills in dashboard Fixes for issues found on last run Updated datatables Fixes for minor bugs Updated with fixes for BS5
Update packages to latest
212b7ed to
b721e04
Compare
droberts-ctrlo
commented
Jun 19, 2026
Cypress tests are failing as they now do not automatically confirm dialogs - when running the tests interactively, they pass fine within both local and container environments.
The aim is to get all tests passing as another PR, including all unit tests that are currently skipped.
@pwlodarski-ctrlo
pwlodarski-ctrlo
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR reviewed - some minor comments added.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this still an up-to-date comment?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this still an up-to-date comment?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could this be a defined constant?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could this remain as $white?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could this use the $white constant?
No description provided.