Skip to content
This repository has been archived by the owner on Feb 2, 2020. It is now read-only.

New feature: timeout value for repeating sentences/intents #3

Closed
koenvervloesem opened this issue Jan 15, 2019 · 3 comments
Closed
Labels
enhancement New feature or request

Comments

@koenvervloesem
Copy link
Owner

Originally asked in #2 by @philipp2310:

Don't allow "old" skills to be repeated:

Forget the stored data after a configurable amount of time (Probably via threads so it is really gone)
This could be used for the other skills as well (You don't want your wife to ask Snips what your last sentence was when you are gone ;))

@koenvervloesem
Copy link
Owner Author

koenvervloesem commented Jan 15, 2019

Great idea! This could be tackled by saving the time of your last utterance and then before repeating it comparing it to the current time and the timeout value the user has configured. However, look at #4 for another (and in my opinion cleaner) approach to solve the issue of your wife asking what your last sentence was.

@koenvervloesem
Copy link
Owner Author

I now read your comment on Discord about the timeout feature:

only thing i want to include that i don't know how: I want the app to forget the last command after ~1 minute and I don't mean just don't respond after that time, but really delete the information

After which Psycho answered threading.Timer.

I include this for reference here.

@koenvervloesem
Copy link
Owner Author

I'm closing this as the app won't be developed further.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant