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

Rezeptschritte vorlesen #24

Closed
langehm opened this issue Nov 10, 2018 · 4 comments
Closed

Rezeptschritte vorlesen #24

langehm opened this issue Nov 10, 2018 · 4 comments

Comments

@langehm
Copy link
Contributor

langehm commented Nov 10, 2018

Als User will ich das mir der Skill die Schritte eines Rezepts vorliest.

Akzeptanzkriterium:

  • Vorlesen der Schritte in richter Reihenfolge
@elodin7 elodin7 changed the title Rezeptschritt vorlesen Rezeptschritte vorlesen Nov 10, 2018
@elodin7 elodin7 self-assigned this Nov 28, 2018
nikoliai added a commit that referenced this issue Dec 3, 2018
@nikoliai nikoliai added the prepraration Issues for starting the project label Dec 3, 2018
nikoliai added a commit that referenced this issue Dec 7, 2018
nikoliai added a commit that referenced this issue Dec 7, 2018
nikoliai added a commit that referenced this issue Dec 7, 2018
@nikoliai
Copy link
Contributor

nikoliai commented Dec 7, 2018

Also es funktioniert soweit. Man muss aber die Intents sehr genau angeben - sonst verwechselt sie Alexa und geht in einen anderen Handler. Für den nächsten Sprint sollten wir ein issue erstellen, alle Ausdrücke zu "säubern", also für jeden Intent eindeutige und einzigartige Phrasen auszudenken.

@nikoliai nikoliai added ready for review and removed prepraration Issues for starting the project labels Dec 7, 2018
@elodin7
Copy link
Contributor

elodin7 commented Dec 8, 2018

Funktion funktioniert. Allerdings würde ich die Methode recipeToRead abändern.
Diese Funktionen könnten wir gleichzeitig nutzen und als savedRecipe umbennen. Somit kann jeder auf das derzeit ausgewählte Rezept zugreifen und handeln.

@elodin7
Copy link
Contributor

elodin7 commented Dec 8, 2018

Funktioniert! Bitte noch testen und Sequenzdiagramm commiten.

nikoliai added a commit that referenced this issue Dec 10, 2018
@elodin7
Copy link
Contributor

elodin7 commented Dec 10, 2018

Intent funktioniert und Diagramm ist vorhanden.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants