Skip to content

Ncrs 4107 upgrade nhsuk react components extensions to react 19#43

Merged
ricksmith130 merged 20 commits intomainfrom
NCRS-4107-Upgrade-nhsuk-react-components-extensions-to-react-19
Oct 15, 2025
Merged

Ncrs 4107 upgrade nhsuk react components extensions to react 19#43
ricksmith130 merged 20 commits intomainfrom
NCRS-4107-Upgrade-nhsuk-react-components-extensions-to-react-19

Conversation

@ricksmith130
Copy link
Contributor

Updated to react 19, including masked input replacement as deprecated

Copy link
Contributor

@CalMitchellBJSS CalMitchellBJSS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The storybook examples of disabled and read-only InputMask don't act as intended but I'm otherwise happy to go ahead and address these later

@ricksmith130 ricksmith130 force-pushed the NCRS-4107-Upgrade-nhsuk-react-components-extensions-to-react-19 branch from 38cae82 to a8f90cf Compare October 15, 2025 12:21
@ricksmith130 ricksmith130 force-pushed the NCRS-4107-Upgrade-nhsuk-react-components-extensions-to-react-19 branch from a8f90cf to 8f1b9f1 Compare October 15, 2025 12:25
- Refactored ProgrammaticControl story to be a proper React component instead of using hooks in render function
- Fixed typescript:S64 violation where useState was called in a non-component function
- Removed unused right: 100px CSS property from headerWithLogo.scss

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@ricksmith130 ricksmith130 force-pushed the NCRS-4107-Upgrade-nhsuk-react-components-extensions-to-react-19 branch 2 times, most recently from 5eb2640 to 68c0950 Compare October 15, 2025 12:29
@sonarqubecloud
Copy link

@CalMitchellBJSS CalMitchellBJSS self-requested a review October 15, 2025 12:33
@ricksmith130 ricksmith130 added this pull request to the merge queue Oct 15, 2025
Merged via the queue into main with commit dc04963 Oct 15, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments