Logsout #6

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.

Sign up for GitHub

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

Jump to bottom

Open

jkeefe wants to merge 2 commits into master from logsout

Contributor

jkeefe commented Oct 3, 2017

Went through a bunch of spots to pull out console.logs ... helping to make sure we're not storing user content unnecessarily.

John Keefe added 2 commits

October 3, 2017 16:26


          took out event logging

7c7586f


          added datetime as sessionID for api.ai

03e5559

Contributor

chriszarate commented Oct 3, 2017

👍 but might be better to put this behind a flag?

Contributor Author

jkeefe commented Oct 4, 2017

What does that mean?! :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet