search

omegabytes / askTheDM

Alexa skill: Get basic info about DND spells, items, and rules
Other
15 stars 10 forks source link

Create Utterance Collision Tool #144

Closed omegabytes closed 6 years ago

omegabytes commented 6 years ago

With the addition of IndexIntent, we may begin running into a problem where the same utterance is used by two or more intents. For example:

IndexIntent get information on {Index} SpellsIntent get information on {Spell}

This doesn't seem to have happened yet, but I don't have a quick way of checking. As we build out the utterances for IndexIntent it is very likely that we will introduce some collisions between intents. I think this is a great use case for your regex know-how.

Eldres commented 6 years ago

As long as we are mindful with our utterances, because we have the "LIST_OF_X" they shouldn't conflict with each other. Will need further testing to make sure this assumption is accurate.

omegabytes commented 6 years ago

I think you're correct about the slots precluding collisions. Since we haven't encountered any collisions over the past few releases, I'm going to mark this as resolved.