Skip to content

Fix android visitor info #82

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

Merged
merged 1 commit into from
May 5, 2021

Conversation

padupuy
Copy link

@padupuy padupuy commented Oct 14, 2020

According to this strange comment on the offical Zendesk Forum and this support post, there is an additionnal config to enable visitor info to be shown on the zendesk dashboard.

Also, bump android zendesk dependencies and add zendeskAppId to the init method to match the last Zendesk Chat SDK v2 documentation.

@fbartho
Copy link
Contributor

fbartho commented Oct 14, 2020

@padupuy -- this appears to duplicate the contents of PR #77 -- would you mind remaking this PR to only include stuff not from PR #77?

I'm about to merge that branch, so if you reapply your changes on a new branch based off of that master, that would be great.

Thanks for contributing!

@padupuy
Copy link
Author

padupuy commented Oct 14, 2020

Sure, tomorrow morning 👌

@fbartho
Copy link
Contributor

fbartho commented Oct 19, 2020

@padupuy bumping this if you're still interested!

@padupuy padupuy force-pushed the fix/android-visitor-info branch from c86a976 to 7788cb7 Compare October 20, 2020 06:57
@padupuy padupuy changed the title Fix android visitor info and bump android dependencies Fix android visitor info Oct 20, 2020
@padupuy
Copy link
Author

padupuy commented Oct 20, 2020

@fbartho sorry for the delay, the PR is now sync with the main branch

@fbartho
Copy link
Contributor

fbartho commented Oct 28, 2020

This code is available with other changes in https://www.npmjs.com/package/react-native-zendesk-chat/v/0.4.1-beta.4

@padupuy
Copy link
Author

padupuy commented Nov 4, 2020

@fbartho the 0.4.1-beta.4 is OK for me on Android and iOS 👌

@fbartho
Copy link
Contributor

fbartho commented Jan 21, 2021

This has been merged into the release branch, and shipped in https://www.npmjs.com/package/react-native-zendesk-chat/v/0.4.1-beta.7

@fbartho fbartho merged commit 7788cb7 into taskrabbit:main May 5, 2021
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