-
Notifications
You must be signed in to change notification settings - Fork 332
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(calling): port calling code into next #3040
Conversation
Co-authored-by: Kesava Krishnan Madavan <mkesavan13@gmail.com> Co-authored-by: Priya Kesari <pkesari@cisco.com>
Co-authored-by: Priya Kesari <pkesari@cisco.com>
Co-authored-by: Dipanshu Sharma <dipashar@cisco.com>
@@ -37,17 +38,34 @@ | |||
"envify" | |||
] | |||
}, | |||
"scripts": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why did we need these changes here ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Kesari3008 we weren't building the webex package so far in the next
branch. We were only building the remaining plugins and testing deployments to npm. As part of this change, we will be building the webex package as well. This was needed in order to get the calling bundle built as well.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we don't have these scripts, we observe the following error when doing yarn run build
at webex-js-sdk level. Without this, the *.min.js
files do not get created.
ERROR in ./packages/webex/src/calling.js 11:0-47
Module not found: Error: Can't resolve '@webex/calling' in '/Users/sreenara/Stuff/Repos/webex-js-sdk/packages/webex/src'
resolve '@webex/calling' in '/Users/sreenara/Stuff/Repos/webex-js-sdk/packages/webex/src'
Parsed request is a module
COMPLETES # NA
This pull request
Brings in the calling code into the next branch for the upcoming release of the Public Calling SDK.
by making the following changes
packages/calling
frompackges/calling/calling
babel.config.js
underpackages/webex
and update package.json to start building thewebex
package.webpack.config.js
to include build of calling.min.jstooling/babel-plugin-inject-package-version.js
by putting in a dummy string for version for now. This is a TODO item.Change Type
The following scenarios where tested
< ENUMERATE TESTS PERFORMED, WHETHER MANUAL OR AUTOMATED >
I certified that
I have read and followed contributing guidelines
I discussed changes with code owners prior to submitting this pull request
I have not skipped any automated checks
All existing and new tests passed
I have updated the documentation accordingly
Make sure to have followed the contributing guidelines before submitting.