Page MenuHomePhabricator

Updated the docs so chatbots can use the Conduit API
ClosedPublic

Authored by jcox on Aug 24 2016, 8:25 PM.
Tags
None
Referenced Files
F15509511: D16443.diff
Wed, Apr 16, 1:02 PM
F15498457: D16443.id.diff
Sun, Apr 13, 12:59 PM
F15443027: D16443.id39556.diff
Thu, Mar 27, 1:44 AM
F15433334: D16443.id39548.diff
Mon, Mar 24, 10:38 PM
F15429657: D16443.id.diff
Mon, Mar 24, 3:30 AM
F15427655: D16443.diff
Sun, Mar 23, 3:24 PM
F15399797: D16443.diff
Mar 17 2025, 7:43 AM
F15388810: D16443.id.diff
Mar 15 2025, 4:26 AM
Subscribers

Details

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
Branch
ChatBotDocFix (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 13431
Build 17256: Run Core Tests
Build 17255: arc lint + arc unit

Event Timeline

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)
epriestley added a reviewer: epriestley.

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.

resources/chatbot/example_config.json
17–18

This can probably be removed, too.

This revision is now accepted and ready to land.Aug 24 2016, 10:54 PM

(That is, old login was user + cert, new login is token-only, I believe.)

jcox edited edge metadata.
  • Removed references to conduit.user from the docs.
This revision was automatically updated to reflect the committed changes.