Skip to content
This repository was archived by the owner on Jan 14, 2024. It is now read-only.

London class 8-Mohammad Alamin - JS - week 1 - #259

Closed
Alamin-eng wants to merge 4 commits into
CodeYourFuture:masterfrom
Alamin-eng:master
Closed

London class 8-Mohammad Alamin - JS - week 1#259
Alamin-eng wants to merge 4 commits into
CodeYourFuture:masterfrom
Alamin-eng:master

Conversation

@Alamin-eng

@Alamin-eng Alamin-eng commented Dec 16, 2021

Copy link
Copy Markdown

Volunteers: Are you marking this coursework? You can find a guide on how to mark this coursework in HOW_TO_MARK.md in the root of this repository

Your Details

  • Your Name: Mohammad Alamin
  • Your City: London
  • Your Slack Name: Mohammad Alamin - trainee

Homework Details

  • Module: JS
  • Week: 1

Notes

  • What did you find easy? Exercises and mandatory files

  • What did you find hard? figuring out the jest errors that exist in many files

  • What do you still not understand? Jest

  • Any other notes? Overall straight forward Java script exercises but many errors exits in original repo


View rendered README.md

@Alamin-eng

Alamin-eng commented Dec 16, 2021

Copy link
Copy Markdown
Author

@haack All mandetory tests I passed in my Vs code but I don't know why its showing 1 test is not passing, And I was unable to run any test for all files in extra folder, Tried upgrading the jest version and all but no luck.

Comment thread README.md
@@ -2,7 +2,7 @@

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Well done for completing the Java Script Week 1.

@@ -1,3 +1,5 @@
// Start by creating a variable `message`
let message = 'Its cold outside!'

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

You have a spelling mistake here, instead of Its use It's please.When iI rush I do the same mistake :)

@@ -1,3 +1,5 @@
// Start by creating a variable `message`
let message = 'Its cold outside!'
let messegeType = typeof message

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

You are missing a semicolon here.

// Start by creating a variable `message`
let message = 'Its cold outside!'
let messegeType = typeof message
console.log(`${message}. It's a ${messegeType} type message`);

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Very good understanding of interpolation since in this stage.

let myName = 'Mohammad '
let nameLength = myName.length
let trimName = myName.trim()
let message = 'My name is '+trimName+' '+'and my name is '+nameLength+' characters long'

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

To simplify your code instead of using a new string to create space you van use a space inside of the quotation marks.

@@ -1,5 +1,10 @@
// Declare your function first
let divide = function(a, b){

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Very good understanding of declaring and assigning but in this case you can simplify your code just by declaring the function strait away function divde(){
}

@@ -5,8 +5,11 @@
Write a function that converts a price to USD (exchange rate is 1.4 $ to £)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Well done on completing the extra exercises 👍



function concatenate(firstWord, secondWord, thirdWord) {
return `${firstWord} ${secondWord} ${thirdWord}`

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Very good understanding of interpolation but in this exercise we are being asked to use the concat() method. Remember concat() doesn't add spaces between strings in this case we have to use the " " to add space.

Comment thread package.json
"jest-extended": "^0.11.5"
"jest-extended": "^1.2.0"
}
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

You have done a great job Alamin 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants