Skip to content

Conversation

@azizomarck
Copy link

No description provided.

button {
font-family: Helvetica;
font-size: 10pt;
/*width: 92px;*/
Copy link
Contributor

Choose a reason for hiding this comment

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

Always delete commented out code

font-size: 10pt;
/*width: 92px;*/
}
textarea {
Copy link
Contributor

Choose a reason for hiding this comment

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

indenting is a big deal when css gets really big, make sure that your indentations are always proper

*/

const findSimillarQuestion = givenInput => (item) => {
// return item.input.toUpperCase().includes(givenInput.toUpperCase());
Copy link
Contributor

Choose a reason for hiding this comment

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

same comment for deleting commented out code

Copy link
Contributor

Choose a reason for hiding this comment

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

also why are you no longer using toUpperCase?



// repaly function
const replay = () => {
Copy link
Contributor

Choose a reason for hiding this comment

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

spelling

bot = detectedQuestion.output[rndmNumber];
}

} else if (question == "show me a dog") {
Copy link
Contributor

Choose a reason for hiding this comment

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

would be good if you abstracted this further and instead of checking specifically for these 2 strings you adding these questions as part of the data array

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