-
Notifications
You must be signed in to change notification settings - Fork 101
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
Add permanent option context var #67
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good @medtaher123 Thanks 👏
I think you will need to adapt the unit tests.
|
@medtaher123 Thank you 🙏 Can you please add a comment to the field permanent to explain what it does ? |
Motivation
We need to add a flag for context variables called "permanent" boolean. Currently a context var value is only kept during the current conversation and then gets lost. It would be good to add the "permanent" option allow the value to be stored in the subscriber object in a attribute called "context" (json).
Feature desciption:
There are two types of context variables in Hexabot: non-permanent and permanent.
1. Non-Permanent Context Variables
2. Permanent Context Variables
Fixes #48
Type of change
Please delete options that are not relevant.
Checklist: