Skip to content
This repository has been archived by the owner on Apr 8, 2020. It is now read-only.

Change ChatEngine#create to ChatEngineCore#create #337

Closed
wants to merge 2 commits into from
Closed

Change ChatEngine#create to ChatEngineCore#create #337

wants to merge 2 commits into from

Conversation

samiahmedsiddiqui
Copy link
Contributor

I think, it should be linked to https://www.pubnub.com/docs/chat-engine/global#chatenginecore as the ChatEngineCore is the Global Object to create the ChatEngine instance..

I think, it should be linked to https://www.pubnub.com/docs/chat-engine/global#chatenginecore as the ChatEngineCore is the Global Object to create the ChatEngine instance..
@ianjennings
Copy link
Contributor

I believe this pull needs master to be merged into it in order for it to pass tests.

Copy link
Contributor

@ianjennings ianjennings left a comment

Choose a reason for hiding this comment

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

LGTM. Merged master in, hope that makes tests pass.

@ianjennings
Copy link
Contributor

Hrm, tests still not passing. It's trying to make requests to:
https://pubsub.pubnub.com/v1/blocks/sub-key/undefined/chat-engine-server

@ianjennings
Copy link
Contributor

This was outdated so I duplicated it in another pull.
#354

@samiahmedsiddiqui samiahmedsiddiqui deleted the patch-3 branch April 24, 2018 06:09
@samiahmedsiddiqui
Copy link
Contributor Author

Thanks @ianjennings 😄

@ianjennings
Copy link
Contributor

You got it @samiahmedsiddiqui :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants