Updated the docs so chatbots can use the Conduit API
ClosedPublic
Actions

Authored by jcox on Aug 24 2016, 8:25 PM.

Details

Reviewers

epriestley

Group Reviewers

Blessed Reviewers

Commits

rP8cdf1a890a01: Updated the docs so chatbots can use the Conduit API

Summary

Previously, the chatbot docs instructed users to get certificates for the conduit API and put the cert in a conduit.cert config key. In order to get the chatbot to work, I needed to instead get an API key and put it in the conduit.token config entry.

Test Plan

Doc fix. Tried the new documented way and it worked.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

jcox updated this revision to Diff 39548.Aug 24 2016, 8:25 PM

jcox retitled this revision from to Updated the docs so chatbots can use the Conduit API.

jcox updated this object.

jcox edited the test plan for this revision. (Show Details)

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptAug 24 2016, 8:25 PM

Herald added a subscriber: epriestley. · View Herald Transcript

I think you can drop all references to conduit.user too if you want, should be fully obsoleted by conduit.token.

You could also reasonably remove the code which actually uses conduit.user, although this probably isn't worth the hassle since it'll probably break some users. Some vaguely related discussion at the bottom of T4377.