Skip to content

Update fundamentals.md#56

Merged
Oluwasetemi merged 4 commits intoOluwasetemi:mainfrom
Aycrescendo:issue-#47-Code-block-formatting
Jan 20, 2026
Merged

Update fundamentals.md#56
Oluwasetemi merged 4 commits intoOluwasetemi:mainfrom
Aycrescendo:issue-#47-Code-block-formatting

Conversation

@Aycrescendo
Copy link
Copy Markdown
Contributor

Code block bocking issue

@netlify
Copy link
Copy Markdown

netlify Bot commented Jan 8, 2026

Deploy Preview for javascript-note ready!

Name Link
🔨 Latest commit 90c1bc0
🔍 Latest deploy log https://app.netlify.com/projects/javascript-note/deploys/6962baecf47ddc0008fd0c6e
😎 Deploy Preview https://deploy-preview-56--javascript-note.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Comment thread pages/fundamentals.md Outdated

```js {monaco-run} {autorun: false}
// alert(true || false); // alert('Boy' || 4)
alert(true || false); alert('Boy' || 4)
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Prefer console.log() over alert on this line

Comment thread pages/fundamentals.md Outdated
if (size == 20) {
// alert('Your size is 20')
// alert('Thank you')
alert('Your size is 20')
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Prefer console.log over alert here. and the line below it.

Comment thread pages/fundamentals.md Outdated
// alert('Your size is 20')
// alert('Thank you')
alert('Your size is 20')
alert('Thank you')
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Prefer console.log over alert here.

@Oluwasetemi
Copy link
Copy Markdown
Owner

You will need to pull in the latest update merged into your fork.

If you are working on the web, you needs to sync your fork with the main project. When you are within your work, locate the commits and look for sync fork with the main.

If you are working on the local, your IDE - You should pull from the upstream(Oluwasetemi/react-note).

git remote add upstream https://github.com/Oluwasetemi/react-note
git pull upstream main

You will have to fix the conflict on your IDE, accept incoming changes and later ensure your fixes are still present.

Do ask any question for clarification.

@Oluwasetemi Oluwasetemi merged commit e3772de into Oluwasetemi:main Jan 20, 2026
5 checks passed
@Oluwasetemi
Copy link
Copy Markdown
Owner

Fixed this for you

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.

Fix: Code block formatting issues on page 14 (Logical operators section)

2 participants