Skip to content
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

Release new demo app and SDK improvements. #119

Merged
merged 38 commits into from
May 22, 2023
Merged

Conversation

alonso-cadenas
Copy link
Collaborator

@alonso-cadenas alonso-cadenas commented May 22, 2023

Description

Jira Ticket: CALL-3749

Merge Checklist

Q A
Adds Documentation?
Any Dependency Changes?
Patch: Bug Fix?
Minor: New Feature?
Major: Breaking Change?

BRAVE Checklist

  • I have read the BRAVE checklist and confirmed if the following is necessary.
Q A
Backwards Compatible?
Rollout/Rollback Plan?
Automated test coverage? Unit tests
Verified that changes work?
Expect Dependencies to Fail?

esme and others added 30 commits April 3, 2023 11:45
* Create demo app UI using React and TS
* Deploy React app to GitHub Pages

* Set up webpack config and node scripts to output demo apps in common build folder

* Configure gh-pages in calling-extensions-sdk
* Set up unit test framework and automate tests

* Deploy to gh pages automatically

* Separate workflows for previews and deploys

* Use npm ci

* Add cross-env
* Add minimal phone number validation to keypad

* Add test
* Update event handling console log messages

* Move logging to iframe manager
* Add developer console banner

* Preview all branches with demo keyword

* Close widget after saving call
* Add icons to demo app
* Backfill unit tests

* Fix createEngagement called twice (#87)
* Add fake numbers to the From Number dropdown
* Use latest package version

* Temporarily comment out imports for local dev

* Update patch dependencies

* Make errorType import easier
* Organize NPM scripts in the demo app
* Merge changes from master branch

* Rename __tests__ to dist-test
* Add documentation for demo widgets
* Use 0.0.15-alpha.1

---------

Co-authored-by: Alonso Cadenas <cadenas.alonso@gmail.com>
* Rename demo widgets
* Fix build warnings and iFrame uninitialized message

* Optimize config for dev or prod build
* Send callStartTime in outgoingCall message
* Specify minimum required Node.js version to maintain compatibility with dependencies

* Remove node_options
* Enforce order of events in demo minimal js
alonso-cadenas and others added 5 commits May 10, 2023 13:42
* Add missing events

* Finalize demo app documentation

* Update diction

* Omit line number
* Use local value to install demo widget

* Fix spacing
* Simplify publish scripts and add prerelease scripts.

* pre-releases should be published as alpha versions

* 0.0.15-alpha.2

* 0.0.15-alpha.3

* 0.0.15-beta.0

* 0.1.0-0

* 0.1.0-alpha.0

* 0.1.0-beta.0

* prepatch already exists via alpha and beta commands
@alonso-cadenas alonso-cadenas marked this pull request as ready for review May 22, 2023 16:52
@alonso-cadenas alonso-cadenas requested a review from a team as a code owner May 22, 2023 16:52
@alonso-cadenas alonso-cadenas requested a review from esme May 22, 2023 16:52
* Enable github actions

* Use latest sdk version in demos

* Set versions for demos
* Update demo docs

* Enable branch previews
* test gha

* Enable github action deploy previews

* Revert "test gha"

This reverts commit ca86ab0.
@github-actions
Copy link

PR Preview Action v1.4.4
🚀 Deployed preview to https://HubSpot.github.io/calling-extensions-sdk/pr-preview/pr-119/
on branch gh-pages at 2023-05-22 21:16 UTC

Copy link
Contributor

@esme esme left a comment

Choose a reason for hiding this comment

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

🚀

@alonso-cadenas alonso-cadenas merged commit d108746 into master May 22, 2023
@alonso-cadenas alonso-cadenas deleted the project-demo-v1 branch May 22, 2023 21:25
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.

2 participants