Skip to content

fix(curriculum): clear up confusion of array index example#45576

Merged
naomi-lgbt merged 1 commit intofreeCodeCamp:mainfrom
sprajams:patch-2
Mar 30, 2022
Merged

fix(curriculum): clear up confusion of array index example#45576
naomi-lgbt merged 1 commit intofreeCodeCamp:mainfrom
sprajams:patch-2

Conversation

@sprajams
Copy link
Copy Markdown
Contributor

@sprajams sprajams commented Mar 30, 2022 •
edited
Loading

Checklist:

  • I have read freeCodeCamp's contribution guidelines.
  • My pull request has a descriptive title (not a vague title like Update index.md)
  • My pull request targets the main branch of freeCodeCamp.
  • I have tested these changes either locally on my machine, or GitPod.

Closes #45572

@gitpod-io
Copy link
Copy Markdown

gitpod-io bot commented Mar 30, 2022

@github-actions github-actions bot added the scope: curriculum Lessons, Challenges, Projects and other Curricular Content in curriculum directory. label Mar 30, 2022
@ghost
Copy link
Copy Markdown

ghost commented Mar 30, 2022

👀 Review this PR in a CodeSee Review Map

Preview the Review Map

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map Legend

naomi-lgbt
naomi-lgbt approved these changes Mar 30, 2022
View reviewed changes
Copy link
Copy Markdown
Member

@naomi-lgbt naomi-lgbt left a comment

Choose a reason for hiding this comment

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

This looks good to me.

@naomi-lgbt naomi-lgbt enabled auto-merge (squash) March 30, 2022 17:40
@sprajams
Copy link
Copy Markdown
Contributor Author

sprajams commented Mar 30, 2022

Thank you again @nhcarrigan!

gikf
gikf approved these changes Mar 30, 2022
View reviewed changes
Copy link
Copy Markdown
Member

@gikf gikf left a comment

Choose a reason for hiding this comment

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

Looks good 👍

@naomi-lgbt naomi-lgbt merged commit 4056752 into freeCodeCamp:main Mar 30, 2022
@gikf
Copy link
Copy Markdown
Member

gikf commented Mar 30, 2022

Hi @sprajams. Congrats on your first pull request (PR)! 🎉

Thank you for your contribution to the page! 👍
We are happy to accept these changes and look forward to future contributions. 📝

@sprajams
Copy link
Copy Markdown
Contributor Author

sprajams commented Mar 30, 2022

Hi @gikf, thank you for the feedback and I am very excited to continue contributing! 😄

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

Reviewers

@gikf gikf gikf approved these changes

@naomi-lgbt naomi-lgbt naomi-lgbt approved these changes

Assignees

No one assigned

Labels

scope: curriculum Lessons, Challenges, Projects and other Curricular Content in curriculum directory.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Basic Javascrip - Access Array Data with Indexes Example is confusing.

3 participants

@sprajams @gikf @naomi-lgbt