Skip to content

Adding sample for hosting WebChat in an Angular application#1813

Merged
corinagum merged 4 commits intomasterfrom
omsour/ng_webchat_sample_0.1
Mar 14, 2019
Merged

Adding sample for hosting WebChat in an Angular application#1813
corinagum merged 4 commits intomasterfrom
omsour/ng_webchat_sample_0.1

Conversation

@omarsourour
Copy link
Copy Markdown
Contributor

Fixes #1423

@corinagum
Copy link
Copy Markdown
Contributor

Looks good to me! Can you add an entry to CHANGELOG.md? Thanks for all the help.

@coveralls
Copy link
Copy Markdown

coveralls commented Mar 14, 2019

Coverage Status

Coverage remained the same at 53.637% when pulling 33d3d91 on omsour/ng_webchat_sample_0.1 into 7dcdacb on master.

@omarsourour omarsourour force-pushed the omsour/ng_webchat_sample_0.1 branch from e402bc9 to 36cced6 Compare March 14, 2019 19:25
@omarsourour
Copy link
Copy Markdown
Contributor Author

@corinagum Done, added entry in ChangeLog. Please give it a look :)

@corinagum
Copy link
Copy Markdown
Contributor

Oops! Sorry Omar, a couple more things

  • could you update the samples list on the root README.md
  • update samples/README.md with same change

Otherwise, looks good and I will merge after your final commit. Sorry about that!

@omarsourour
Copy link
Copy Markdown
Contributor Author

@corinagum Done :) .. Please take a look

Also, may I ask/suggest why not have the root README.md link to the samples/README.md? :)

@corinagum
Copy link
Copy Markdown
Contributor

@omarsourour oh is there a way to automatically update one and it will update the other? If so please let me know how!

Thank you!

@omarsourour omarsourour reopened this Mar 14, 2019
@corinagum corinagum merged commit 9c217f2 into master Mar 14, 2019
@corinagum corinagum deleted the omsour/ng_webchat_sample_0.1 branch April 15, 2019 22:35
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