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

deleted by mistake #298

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

deleted by mistake #298

wants to merge 2 commits into from

Conversation

smily342
Copy link

Netlify link

Add your Netlify link here.
PS. Don't forget to add it in your readme as well.

Sorry it was deleted by mistake

Copy link
Contributor

@JennieDalgren JennieDalgren left a comment

Choose a reason for hiding this comment

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

You are up to something good here!
But it feels like the chatbot isnt really finished.
I don't understand the flow of the questions and there isn't really any proper end to the bot.

My suggestion is to create proper named functions for all the steps in the flow, also make sure to follow your plan - plan your userflow with pen and paper (or a digital version of pen and paper)
Or just follow the exact same flow as the pizza bot from last week, but do it in the browser this time.

Don't hesitate to reach out if you have any questions! We are here to help.

@smily342
Copy link
Author

smily342 commented Oct 6, 2024

Please look at my last update :)

Copy link
Contributor

@JennieDalgren JennieDalgren left a comment

Choose a reason for hiding this comment

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

Great job Brian!
🚀

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