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

Holiday bot #290

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

Holiday bot #290

wants to merge 20 commits into from

Conversation

zoe-lindqvist
Copy link

Copy link
Contributor

@HIPPIEKICK HIPPIEKICK 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 on implementing this holiday bot 🥳 Let's have a look at some feedback.

JavaScript

  • Smooth flow of your chatbot, with appropriate prompts and responses ⭐
  • Great job breaking the code into modular functions (greeting, showMoodOptions, etc.). This makes the chatbot’s logic easier to follow and maintain. 👏

Clean Code

  • The code is generally well-organized with meaningful variable names, keep this up!

Keep up the good work 🥳

Comment on lines +23 to +28
const botReply = (msg) => {
setTimeout(() => showMessage(msg, "bot"), 500);
};
const userReply = (msg) => {
showMessage(msg, "user");
};
Copy link
Contributor

Choose a reason for hiding this comment

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

Nice ⭐


// Functions goes here 👇
// Variables
let questionNumber = 1;
Copy link
Contributor

Choose a reason for hiding this comment

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

Really good idea to keep track of the flow ⭐

Copy link

@Fannyhenriques Fannyhenriques left a comment

Choose a reason for hiding this comment

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

Wow! I loved your holiday-chat-bot! It really got me in the mood for a vacation! 😊

Everything works smoothly and the background image and all the emojis are a really nice touch! They really make the experience more fun! I also loved the inclusion of activity suggestions with links.

Your functions are well-structured, keeping the code clean and easy to follow. Great job!

Also, great work with the setTimeout, the conversation has an overall flow that feels very user-friendly. And - great move with the start-over-button! I wish I had thought of that in my own project! 😊

All in all, a fantastic chat-bot and great use of JavaScript! SUPER impressed with your work!

botReply(`${choice} sounds fun! Are you ready to start planning your trip?`);

// Add suggestions based on the selected activity
let suggestions = "";

Choose a reason for hiding this comment

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

nice!!

setTimeout(() => {
inputWrapper.innerHTML = `
<button id="start-over">Start Over</button>
`;

Choose a reason for hiding this comment

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

Loved this!

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.

3 participants