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

Include JSQMessagesViewController in public headers #137

Merged
merged 1 commit into from
Nov 30, 2017

Conversation

cheungpat
Copy link
Contributor

This allows projects using SKYKitChat to use JSQ classes,
which is now embedded in the SKYKitChat.

refs #136

@cheungpat
Copy link
Contributor Author

This is a partial fix for the problem you are having @tensiuyan

@cheungpat
Copy link
Contributor Author

checkout this branch to evaluate this pull request using the demo project https://github.com/cheungpat/SkygearChat-iOS-Demo/tree/fix-for-1.2.0

@Steven-Chan
Copy link
Contributor

checkout this branch to evaluate this pull request using the demo project https://github.com/cheungpat/SkygearChat-iOS-Demo/tree/fix-for-1.2.0

@cheungpat

I've just tested with the demo, and it failed to build because the chat sdk in pod cannot find the jsq classes.

I find the source code of jsq classes are not found in the pods folder.

@Steven-Chan
Copy link
Contributor

The demo is working after adding a submodule flag in pod file. It may be because the demo is not importing a released version. So i would merge this and make a new release first.

@Steven-Chan Steven-Chan merged commit 069b62f into SkygearIO:master Nov 30, 2017
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.

None yet

2 participants