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

Make it simpler to override system logic adapters. #1296

Merged
merged 1 commit into from
May 7, 2018
Merged

Conversation

gunthercox
Copy link
Owner

@gunthercox gunthercox commented May 4, 2018

This makes a change so that developers can override the built in set of system logic adapters that ChatterBot uses.

This isn't something that is needed for typical use of ChatterBot but in some advanced cases it can be useful to disable them.

Example usage

chatbot = ChatBot(
    system_logic_adapters=[]
)

For #1273

This makes a change so that developers can create a subclass of
ChatBot that overrides the build in set of system logic adapters.

This isn't something that is needed for typical use of ChatterBot
but in some advanced cases it can be useful to disable them.
Copy link
Collaborator

@vkosuri vkosuri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 💯 👍

@gunthercox gunthercox merged commit 84ce86f into master May 7, 2018
@gunthercox gunthercox deleted the GH-1273 branch June 13, 2018 17:32
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