Skip to content
This repository has been archived by the owner on May 3, 2021. It is now read-only.

Request: forbid UnknownWord & remove last entry #11

Closed
n0o0ne opened this issue Apr 3, 2019 · 2 comments
Closed

Request: forbid UnknownWord & remove last entry #11

n0o0ne opened this issue Apr 3, 2019 · 2 comments
Assignees
Labels
bug Something isn't working

Comments

@n0o0ne
Copy link

n0o0ne commented Apr 3, 2019

It would be nice that "UnknownWord" can't be added to your list. To pronounce it in german for removal is nearly impossible and if it is on your list it has no use. Because of the hassle to remove UnknownWord from my list i suggest a command to remove the last added entry would be a useful feature if the word is wrongly recognised. If bring! does not count maybe it's possible to combine it with the repeat the last command app.

@philipp2310 philipp2310 self-assigned this Apr 7, 2019
@philipp2310 philipp2310 added the bug Something isn't working label Apr 7, 2019
@philipp2310
Copy link
Owner

Strangely I am not able to reproduce the "UnknownWord" issue.
Either I get something wrong added to the list, or I am asked "What do you want to add".
Is it some special word that it catches as UnknownWord and really uses as a slot value?

Regarding "undo last action", it would be awesome if it was possible to include it in "Wie Bitte?"/"Say it again". But unfortunately the undo action would be something different for every app/intent. I will keep it on my list, maybe we could agree on an interface that adds an "undo" command to the last heard request - then every app developer can implement it without any collisions in the intend detection. But if there is no specific implementation and it is only a slot value "undo", this would trigger the action again, instead of rolling it back.

As Bring! really isn't storing the time of adding the items, I need to store it in a local list.

I won't have the time to fix it in the next two weeks - but it will be the first task afterwards! Thanks for reporting!

@philipp2310
Copy link
Owner

Context sensitivity is part of Project Alice

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants