Skip to content

Commit

Permalink
One single React package during development-time (#2274)
Browse files Browse the repository at this point in the history
* Remove unnecessary dependencies

* Update entry

* Remove lerna from samples

* Update PR number

* Update entry

* Add samples back to lerna

* Update entry
  • Loading branch information
compulim authored Aug 7, 2019
1 parent ae8f30f commit e4f795a
Show file tree
Hide file tree
Showing 24 changed files with 12,815 additions and 9,732 deletions.
12 changes: 11 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,10 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
## [Unreleased]
- Added missing Norwegian (nb-NO) translations, by [@taarskog](https://github.com/taarskog)

### Breaking changes

- We will no longer include `react` and `react-dom` in our NPM package, instead, we will requires peer dependencies of `react@^16.8.6` and `react-dom@^16.8.6`

### Changed

- `*`: Bumps all dev dependencies to latest version, by [@compulim](https://github.com/compulim), in PR [#2182](https://github.com/microsoft/BotFramework-WebChat/pull/2182), notably
Expand Down Expand Up @@ -51,7 +55,13 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
- [`mixin-deep@1.3.2`](https://www.npmjs.com/package/mixin-deep)
- [`set-value@2.0.1`](https://www.npmjs.com/package/set-value)
- [`union-value@1.0.1`](https://www.npmjs.com/package/union-value)
- Bumps [`web-speech-cognitive-services@4.0.1-master.c12f923`](https://www.npmjs.com/package/web-speech-cognitive-services), by [@compulim](https://github.com/compulim) in PR [#2246](https://github.com/microsoft/BotFramework-WebChat/pull/2246)
- Bumps [`web-speech-cognitive-services@4.0.1-master.ad6e780`](https://www.npmjs.com/package/web-speech-cognitive-services), by [@compulim](https://github.com/compulim) in PR [#2246](https://github.com/microsoft/BotFramework-WebChat/pull/2246) and PR [#2274](https://github.com/microsoft/BotFramework-WebChat/pull/2274)
- Fix for React hooks constraints: both app and component must share the same reference of [`react`](https://www.npmjs.com/package/react) and [`react-dom`](https://www.npmjs.com/package/react-dom), in PR [#2274](https://github.com/microsoft/BotFramework-WebChat/pull/2274)
- `/`: Install [`react`](https://www.npmjs.com/package/react) and [`react-dom`](https://www.npmjs.com/package/react-dom) to `devDependencies`
- `bundle`: Move [`react`](https://www.npmjs.com/package/react) and [`react-dom`](https://www.npmjs.com/package/react-dom) from `dependencies` to `peerDependencies`
- `component`: Remove [`react`](https://www.npmjs.com/package/react) and [`react-dom`](https://www.npmjs.com/package/react-dom) from `devDependencies`
- `playground`: Remove [`react`](https://www.npmjs.com/package/react) and [`react-dom`](https://www.npmjs.com/package/react-dom) from `dependencies`
- `samples/*`: Move to production version of Web Chat, and bump to [`react@16.8.6`](https://www.npmjs.com/package/react) and [`react-dom@16.8.6`](https://www.npmjs.com/package/react-dom)

### Fixed

Expand Down
2 changes: 1 addition & 1 deletion lerna.json
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
{
"lerna": "2.11.0",
"lerna": "3.15.0",
"packages": [
"packages/*",
"samples/12.customization-minimizable-web-chat",
Expand Down
45 changes: 45 additions & 0 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 2 additions & 0 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -102,6 +102,8 @@
"lerna": "^3.15.0",
"lint-staged": "^9.2.0",
"prettier": "^1.18.2",
"react": "^16.8.6",
"react-dom": "^16.8.6",
"selenium-webdriver": "^4.0.0-alpha.4",
"serve-handler": "^6.1.0"
}
Expand Down
60 changes: 9 additions & 51 deletions packages/bundle/package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

8 changes: 5 additions & 3 deletions packages/bundle/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -44,11 +44,9 @@
"memoize-one": "^5.0.2",
"microsoft-speech-browser-sdk": "^0.0.12",
"prop-types": "^15.7.2",
"react": "^16.8.6",
"react-dom": "^16.8.6",
"sanitize-html": "^1.19.0",
"url-search-params-polyfill": "^5.0.0",
"web-speech-cognitive-services": "4.0.1-master.c12f923",
"web-speech-cognitive-services": "4.0.1-master.ad6e780",
"whatwg-fetch": "^3.0.0"
},
"devDependencies": {
Expand All @@ -75,5 +73,9 @@
"webpack": "^4.35.3",
"webpack-cli": "^3.3.5",
"webpack-visualizer-plugin": "^0.1.11"
},
"peerDependencies": {
"react": "^16.8.6",
"react-dom": "^16.8.6"
}
}
34 changes: 0 additions & 34 deletions packages/component/package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 0 additions & 2 deletions packages/component/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -40,8 +40,6 @@
"concurrently": "^4.1.1",
"eslint-plugin-prettier": "^3.1.0",
"prettier": "^1.18.2",
"react": "^16.8.6",
"react-dom": "^16.8.6",
"rimraf": "^2.6.3",
"typescript": "^3.5.3"
},
Expand Down
Loading

0 comments on commit e4f795a

Please sign in to comment.