aBug/10212 refactor unsafe components

[jryanthe4th]

Description

OperationContainer changes:

• The logic from UNSAFE_componentWillReceiveProps has been moved to componentDidUpdate so swagger-ui works with react 18.
• It checks if response has changed and updates executeInProgress accordingly.
• It ensures that requestResolvedSubtree() is only called when isShown changes from false to true.

ContentType changes:

• Removed UNSAFE_componentWillReceiveProps as its logic has been handled properly in componentDidUpdate.
• componentDidUpdate now watches for changes in contentTypes or value.
• If contentTypes change or the current value is not included, it updates the value.
• Improved readability and efficiency by ensuring onChange is only called when necessary.

root-injects.jsx changes:
Fix reactDOM import error:
Warning: You are importing createRoot from "react-dom" which is not supported. You should instead import it from "react-dom/client".

Motivation and Context

Swagger-ui throws errors when used with react 18 due to the deprecation of UNSAFE_componentWillReceiveProps.

#10212

Fixes #10212

How Has This Been Tested?

Ran the app locally and clicked multiple links and buttons

Screenshots (if appropriate):

Checklist

My PR contains...

• No code changes (src/ is unmodified: changes to documentation, CI, metadata, etc.)
• Dependency changes (any modification to dependencies in package.json)
• Bug fixes (non-breaking change which fixes an issue)
• Improvements (misc. changes to existing features)
• Features (non-breaking change which adds functionality)

My changes...

• are breaking changes to a public API (config options, System API, major UI change, etc).
• are breaking changes to a private API (Redux, component props, utility functions, etc.).
• are breaking changes to a developer API (npm script behavior changes, new dev system dependencies, etc).
• are not breaking changes.

Documentation

• My changes do not require a change to the project documentation.
• My changes require a change to the project documentation.
• If yes to above: I have updated the documentation accordingly.

Automated tests

• My changes can not or do not need to be tested.
• My changes can and should be tested by unit and/or integration tests.
• If yes to above: I have added tests to cover my changes.
• If yes to above: I have taken care to cover edge cases in my tests.
• All new and existing tests passed.

[Stanley-Jovel]

Someone merge this please

[robert-hebel-sb]

@jryanthe4th thank you for your contribution and the time spent on refactoring 🙏

A few changes are unrelated to the refactoring of unsafe components. We would appreciate it if you could create separate PRs for these, as it would make it easier to track the changes in the commit history

Addressing the ReactDOM warning would lead to the loss of support for React 16 and 17, given that the client API was introduced in React 18

[Copilot]

Pull Request Overview

This PR refactors legacy lifecycle methods to modern React patterns and fixes a React 18-related import issue.

• Refactors OperationContainer to use componentDidUpdate instead of UNSAFE_componentWillReceiveProps with enhanced checks for state updates.
• Rewrites ContentType to update its state in componentDidUpdate and improves code clarity and efficiency.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
src/core/containers/OperationContainer.jsx	Replaced UNSAFE_componentWillReceiveProps with componentDidUpdate and refined request triggering logic.
src/core/components/content-type.jsx	Replaced unsafe lifecycle method with componentDidUpdate and refined initial value handling.

Comments suppressed due to low confidence (1)

src/core/containers/OperationContainer.jsx:101

• [nitpick] Ensure that isShown is always a boolean; if there's a possibility that it could be a non-boolean truthy value, consider explicitly comparing with 'true' to avoid any unexpected behavior.

if (isShown && resolvedSubtree === undefined && !prevProps.isShown) {