-
Notifications
You must be signed in to change notification settings - Fork 11
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(chat): slotting and streaming #145
feat(chat): slotting and streaming #145
Conversation
…-updates' into feat/chat-storybook-updates
fix(charts): fix some ts errors
✅ Deploy Preview for carbon-labs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
packages/chat/package.json
Outdated
], | ||
"types": "./src/index.d.ts", | ||
"customElements": "custom-elements.json", | ||
"scripts": { | ||
"build": "gulp build --option chat", | ||
"build": "gulp build --option feedback", |
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.
please change back, otherwise it is not building the chat
package
tsconfig.json
Outdated
@@ -27,5 +27,5 @@ | |||
"types": ["mocha", "sinon"] | |||
}, | |||
"include": ["packages/**/*.ts", ".storybook/**/*.ts", "declarations.d.ts"], | |||
"exclude": ["packages/**/__tests__"] | |||
"exclude": ["packages/**/__tests__","node_modules/vega-lite"] |
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.
I don't think we need to add the vega-lite
here, it doesn't work
packages/chat/package.json
Outdated
"build:dist": "rm -rf dist && rollup --config ../../tools/rollup.config.dist.js", | ||
"build:dist:canary": "rm -rf dist && rollup --config ../../tools/rollup.config.dist.js --configCanary", | ||
"postinstall": "ibmtelemetry --config=telemetry.yml" |
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.
please add the postinstall
script back, it's needed for telemetry to work
find . -type f -name '*.scss' -exec sed -i '' "s|${OLD_TOKEN}|${NEW_TOKEN}|g" {} + | ||
done | ||
|
||
echo “All .scss files updated” |
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.
I removed this last time because I'm not sure this script should live here
…g' into slottingandstreaming
Closes #
{{short description}}
Changelog
New
Changed
Removed
Testing / Reviewing
{{ Add descriptions, steps or a checklist for how reviewers can verify this PR works or not }}