!feat: upgrade react-subapp package to react-router v6 #1890
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
|
Looks like test cases are failing |
recent changes, fixed test cases |
smuthya
added a commit
that referenced
this pull request
Jul 13, 2022
* feat: upgrade to react 18, document issues (#1874) * test(CEECORE-2971): fix fe-frameworkLib CSR test and upgrade React v18 (#1876) * test: resolve CEECORE-2971 * feat(CEECORE-2971): upgrade to React v18 * test(CEECORE-2971): remove comments from fe-frameworkLib CSR test Co-authored-by: Mateja Ristic <matejaristic@MatejaR-USMAC22.local> * refactor(CEECORE-3041): Remove method renderToStaticMarkup from frameworkLib and update test (#1883) * test: resolve CEECORE-2971 * feat(CEECORE-2971): upgrade to React v18 * test(CEECORE-2971): remove comments from fe-frameworkLib CSR test * test(CEECORE-2980): refactor test to work with renderToPipeableNodeStream * refactor(CEECORE-2932): replace depricated render methods and unnecessary dependecies in frameworkLib * test(CEECORE-2932): update ssr-framework test to work with renderToPipeableNodeStream * test(CEECORE-2932): remove react-async-ssr references from ssr-framework test * docs: remove react-async-ssr reference from README * docs: remove react-async-ssr reference from README * refactor: remove react-async-ssr from package.json files * refactor: remove react-async-ssr from package.json files * fix: run getTestWritable() within tests and define as an arrow function * remove renderToStaticMarkup and refactor test in subapp-react Co-authored-by: Mateja Ristic <matejaristic@MatejaR-USMAC22.local> Co-authored-by: Mateja Ristic <matejaristic@MatejaR-USMAC22.lan> * major(CEECORE-3087): Upgrade xarc-react-router package to V6 (#1887) * test(CEECORE-2980): refactor test to work with renderToPipeableNodeStream * test(CEECORE-2932): update ssr-framework test to work with renderToPipeableNodeStream * fix: run getTestWritable() within tests and define as an arrow function * Remove redundant getTestWritable defintion * remove redundant Stream import * feat(CEECORE-2971): upgrade to React v18 * test(CEECORE-2980): refactor test to work with renderToPipeableNodeStream * refactor(CEECORE-2932): replace depricated render methods and unnecessary dependecies in frameworkLib * test(CEECORE-2932): update ssr-framework test to work with renderToPipeableNodeStream * remove getTestWritable * remove test subapp * major: upgrade react-router in xarc-react-router package to v6 * update remaining dependencies in xarc-react-router * upgrade dependencies * remove redundant Strema import Co-authored-by: Mateja Ristic <matejaristic@MatejaR-USMAC22.local> * feat(CEECORE-3011) add a basic react v18 based subapp version 2Subapp2 (#1886) * feat(CEECORE-2971): upgrade to React v18 * test(CEECORE-2980): refactor test to work with renderToPipeableNodeStream * refactor(CEECORE-2932): replace depricated render methods and unnecessary dependecies in frameworkLib * test(CEECORE-2932): update ssr-framework test to work with renderToPipeableNodeStream * fix: run getTestWritable() within tests and define as an arrow function * undo test file changes * update subapp2-basic folders * upgrade xarc-react-router dependencies * test(CEECORE-2980): refactor test to work with renderToPipeableNodeStream * test(CEECORE-2932): update ssr-framework test to work with renderToPipeableNodeStream * fix: run getTestWritable() within tests and define as an arrow function * Remove redundant getTestWritable defintion * remove redundant Stream import * feat(CEECORE-2971): upgrade to React v18 * test(CEECORE-2980): refactor test to work with renderToPipeableNodeStream * refactor(CEECORE-2932): replace depricated render methods and unnecessary dependecies in frameworkLib * test(CEECORE-2932): update ssr-framework test to work with renderToPipeableNodeStream * remove getTestWritable * remove test subapp * major: upgrade react-router in xarc-react-router package to v6 * update remaining dependencies in xarc-react-router * upgrade dependencies * remove redundant Strema import * add react-router v6 support * add title * refactor routes file * refactor routes file * update package.json files for xarc-react-router and subapp2 Co-authored-by: Mateja Ristic <matejaristic@MatejaR-USMAC22.local> * refactor(CEECORE-3033): update redux-router-engine with renderToPipeableStream method for SSR (#1884) * feat(CEECORE-2971): upgrade to React v18 * test(CEECORE-2980): refactor test to work with renderToPipeableNodeStream * test(CEECORE-2932): update ssr-framework test to work with renderToPipeableNodeStream * fix: run getTestWritable() within tests and define as an arrow function * remove renderToStaticMarkup and refactor test in subapp-react * replace renderToNodeStream with renderToPipeableStream, adjust tests * remove dangling commas and trailing empty spaces * remove redundant sample app * remove redundant formatting changes * remove redundant formatting changes in redux-router-engine test Co-authored-by: Mateja Ristic <matejaristic@MatejaR-USMAC22.local> Co-authored-by: Mateja Ristic <matejaristic@MatejaR-USMAC22.lan> * [FIX](CEECORE-3051): React 18 - Suspense fallback rendering issue (#1889) * test(CEECORE-2980): refactor test to work with renderToPipeableNodeStream * test(CEECORE-2932): update ssr-framework test to work with renderToPipeableNodeStream * fix: run getTestWritable() within tests and define as an arrow function * Remove redundant getTestWritable defintion * remove redundant Stream import * feat(CEECORE-2971): upgrade to React v18 * test(CEECORE-2980): refactor test to work with renderToPipeableNodeStream * refactor(CEECORE-2932): replace depricated render methods and unnecessary dependecies in frameworkLib * test(CEECORE-2932): update ssr-framework test to work with renderToPipeableNodeStream * remove getTestWritable * remove test subapp * WIP: change renderToPipeableStream implementation * remove formatting changes * cleanup * refactor frameworkLib tests for new render method implementation * remove redundtant hydration tests and add comments Co-authored-by: Mateja Ristic <matejaristic@MatejaR-USMAC22.local> * !feat: upgrade react-subapp package to react-router v6 (#1890) * upgrade and refactor react-subapp to react-router v6 * remove extra empty comment from test expectations * feat(CEECORE-3010): add a basic react v18 based subapp version 1 (#1880) * add subapp-v1 with react 18 suspense feature * add new suspense and transition features * add comments to suspense demo * upgrade subapp V1 packages * refactor framework-lib to workl with react router v6 * refactor subapp1-online-store to work with react router 6 * upgrade and refactor react-subapp to react-router v6 * add FEATURES section to README * add BrowserRouter to the subapp * remove extra empty comment from test expectations * remove test folder * feat(CEECORE-3162): upgrade React 18.0.0 to 18.2.0 (#1891) * minor: upgrade to react 18.2.0 * minor: upgrade react 18.1.0 to 10.2.0 * upgrade remaining react version to 18.2.0 * fix: update dependencies to resolve build issues (#1892) * fix: fix error with ts 4.7.2 version * fix: fix test cases and lint issues * patch: add eslint-plugin-tsdoc as dev dependency * Removed duplicate 'fetch' variable as it is already defined as a built-in global variable * fix: fix error with ts 4.7.2 version * fix: fix lint issues * revert changes in xarc-react-query test * (fix)CEECORE-3183: fix failing test cases (#1893) * fix: fixed lint issue related to react component * fix: failed tests and checks in subapp-react (#1894) * (fix)CEECORE-3183: fix failing test cases * fix failed checks from subapp-react * fix:remove redundant props (#1895) Co-authored-by: Srinath Muthyala <Srinath.reddy.muthyala@walmart.com> Co-authored-by: mristic505 <mateja.ristic@gmail.com> * fix subapp-server failed test (#1896) * fix: update package dependencies to resolve build issues (#1897) Co-authored-by: Srinath Muthyala <Srinath.reddy.muthyala@walmart.com> * fix: update package dependencies to resolve build issues (#1898) Co-authored-by: Srinath Muthyala <Srinath.reddy.muthyala@walmart.com> Co-authored-by: mristic505 <mateja.ristic@gmail.com> Co-authored-by: Mateja Ristic <matejaristic@MatejaR-USMAC22.local> Co-authored-by: Mateja Ristic <matejaristic@MatejaR-USMAC22.lan> Co-authored-by: smuthya <59977748+smuthya@users.noreply.github.com> Co-authored-by: Srinath Muthyala <Srinath.reddy.muthyala@walmart.com>
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.
Description
package.jsonto React Router v6.framework-libto work with React Router 6 breaking changesssr-framework.spec.jsto work with React Router 6 breaking changes.How to test
Navigate to
packages/subapp-react/and runfyn && fun testDocs
For the complete list of breaking changes visit https://reactrouter.com/docs/en/v6