Skip to content

Holiday bot#290

Open
zoe-lindqvist wants to merge 20 commits into
Technigo:mainfrom
zoe-lindqvist:main
Open

Holiday bot#290
zoe-lindqvist wants to merge 20 commits into
Technigo:mainfrom
zoe-lindqvist:main

Conversation

@zoe-lindqvist

Copy link
Copy Markdown

@HIPPIEKICK HIPPIEKICK left a comment

Copy link
Copy Markdown
Contributor

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 thread code/script.js
Comment on lines +23 to +28
const botReply = (msg) => {
setTimeout(() => showMessage(msg, "bot"), 500);
};
const userReply = (msg) => {
showMessage(msg, "user");
};

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nice ⭐

Comment thread code/script.js

// Functions goes here 👇
// Variables
let questionNumber = 1;

Copy link
Copy Markdown
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 ⭐

@Fannyhenriques Fannyhenriques left a comment

Copy link
Copy Markdown

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!

Comment thread code/script.js
botReply(`${choice} sounds fun! Are you ready to start planning your trip?`);

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

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

nice!!

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

Copy link
Copy Markdown

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