Skip to content

Add sample for using different version of React in hosting app via NPM packages#2509

Merged
compulim merged 9 commits intomicrosoft:masterfrom
compulim:sample-hybrid-react
Oct 30, 2019
Merged

Add sample for using different version of React in hosting app via NPM packages#2509
compulim merged 9 commits intomicrosoft:masterfrom
compulim:sample-hybrid-react

Conversation

@compulim
Copy link
Copy Markdown
Contributor

@compulim compulim commented Oct 29, 2019

Changelog Entry

Description

Starting from Web Chat 4.6, we are migrating to React Hooks and requiring React 16.8.6 or up.

In this sample, we will show how to use Web Chat in an hosting app with React version 16.0.0, instead of the required 16.8.6.

Specific Changes

  • Added a new sample 23.a.hybrid-react-npm

  • Testing Added
    • Currently, no tests will be added for samples

Copy link
Copy Markdown
Contributor

@tonyanziano tonyanziano left a comment

Choose a reason for hiding this comment

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

Looks good to me 👍

Comment thread README.md Outdated
Comment thread CHANGELOG.md Outdated
Comment thread samples/23.a.hybrid-react-npm/README.md Outdated
Comment thread samples/23.a.hybrid-react-npm/README.md Outdated
Comment thread samples/23.a.hybrid-react-npm/README.md Outdated
Comment thread samples/23.a.hybrid-react-npm/README.md Outdated
Comment thread samples/23.a.hybrid-react-npm/README.md Outdated
Comment thread samples/23.a.hybrid-react-npm/README.md Outdated
Comment thread samples/23.a.hybrid-react-npm/README.md Outdated
Comment thread samples/23.a.hybrid-react-npm/README.md Outdated
@compulim compulim merged commit 7261d33 into microsoft:master Oct 30, 2019
@compulim compulim deleted the sample-hybrid-react branch October 30, 2019 03:28
@compulim compulim mentioned this pull request Oct 31, 2019
55 tasks
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.

3 participants