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

Update custom-connectors.mdx #13005

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

arunjose696
Copy link

Fixing 2 small bugs in docs

Proposed changes:

The tutorial for custom connectors have a few bugs
1)The name function in custom connector should have an argument the self class as the name method of Input class has an argument. This was mentioned in this community post
https://forum.rasa.com/t/typeerror-name-takes-0-positional-arguments-but-1-was-given-r/42399/5
2)The curl command to test out the API, sends message in a key "message", but the given code sample for custom connector checks for a key "text", so setting the message to none. This would cause some confusion for users first trying custom connector.

  • ...

Status (please check what you already did):

  • added some tests for the functionality
  • updated the documentation
  • updated the changelog (please check changelog for instructions)
  • reformat files using black (please check Readme for instructions)

Fixing 2 small bugs in docs
@CLAassistant
Copy link

CLAassistant commented Jan 28, 2024

CLA assistant check
All committers have signed the CLA.

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