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

Adding text response use case diagrams diagrams #6

Merged
merged 20 commits into from
Sep 21, 2022
Merged

Conversation

iamyume
Copy link
Collaborator

@iamyume iamyume commented Sep 6, 2022

Addressing issue #4 with use case diagrams that show a message flow between the user and the server in a different number of scenarios.

You can view the diagrams at https://github.com/codeforbtv/courtbot-vt-frontend/blob/issues/4-use-cases/docs/use-cases.md

@iamyume iamyume added this to the MVP milestone Sep 6, 2022
@iamyume iamyume self-assigned this Sep 6, 2022
@courtneyfay
Copy link

These all look good to me!

@courtneyfay
Copy link

If we wanted to be totally thorough, we could just add one more diagram for the case "multiple cases found, but user says No for reminder"

@courtneyfay courtneyfay self-requested a review September 20, 2022 23:26
Copy link

@courtneyfay courtneyfay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

happy to merge it in as is. for super thoroughness, you could add in a 6th case (see comment above). i'll leave that up to you!

@iamyume iamyume merged commit 15352ea into main Sep 21, 2022
@iamyume iamyume deleted the issues/4-use-cases branch September 21, 2022 11:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants