Skip to content
This repository was archived by the owner on Sep 25, 2019. It is now read-only.

fix(challenges): a sentence didn't make sense, so I modified it#296

Merged
scissorsneedfoodtoo merged 1 commit intofreeCodeCamp:devfrom
lalov1:fix/text-change
Sep 6, 2018
Merged

fix(challenges): a sentence didn't make sense, so I modified it#296
scissorsneedfoodtoo merged 1 commit intofreeCodeCamp:devfrom
lalov1:fix/text-change

Conversation

@lalov1
Copy link
Copy Markdown
Contributor

@lalov1 lalov1 commented Sep 2, 2018

ISSUES CLOSED: #18046

Description

Reworded information because it didn't make sense beforehand.

Pre-Submission Checklist

  • Your pull request targets the dev branch.
  • Branch starts with either fix/, feature/, or translate/ (e.g. fix/challenge-tests)
  • All new and existing tests pass the command npm test.
  • Use npm run commit to generate a conventional commit message.
    Learn more here: https://conventionalcommits.org/#why-use-conventional-commits
  • The changes were done locally on your machine and NOT GitHub web interface.
    If they were done on the web interface you have ensured that you are creating conventional commit messages.

Checklist:

  • Tested changes locally.
  • Addressed currently open issue (replace XXXXX with an issue no in next line)

Closes freeCodeCamp/freeCodeCamp#18046

@scissorsneedfoodtoo
Copy link
Copy Markdown
Contributor

@lalov1, thank you for taking the time to update this challenge.

Your changes look very good overall, but I think we can further improve the end of the sentence you edited. Since JSON describes the format of the object rather than the object itself, I think the sentence should read something like the following:

Be careful especially when you're accessing and working with external data in the form of a JSON (JavaScript Object Notation) object.

Or perhaps:

Be careful especially when you're accessing and working with external data such as JSON (JavaScript Object Notation).

There are lots of possible ways to word this, so please let me know if you have another suggestion!

@lalov1
Copy link
Copy Markdown
Contributor Author

lalov1 commented Sep 4, 2018

@scissorsneedfoodtoo I think adding "object" at the end should work. Do you want me to fix and resubmit?

@scissorsneedfoodtoo
Copy link
Copy Markdown
Contributor

Sure, that's absolutely fine!

To do that, make sure you're on the right branch with git checkout fix/text-change. Make the changes and add the file with git add challenges/02-javascript-algorithms-and-data-structures/debugging.json. Then you'll need to use git commit --amend to amend your current commit message. It's fine now, so just exit out after that command. Finally, just push back to your branch with git push -f origin fix/text-change and we'll see the change reflected here.

@lalov1
Copy link
Copy Markdown
Contributor Author

lalov1 commented Sep 6, 2018

@scissorsneedfoodtoo I think I made the fix, but let me know if something went wrong...

@scissorsneedfoodtoo scissorsneedfoodtoo merged commit 3d77920 into freeCodeCamp:dev Sep 6, 2018
@scissorsneedfoodtoo
Copy link
Copy Markdown
Contributor

Everything LGTM! 👍 👍

Thank you for taking the time to fix this issue, @lalov1. Looking forward to your next contribution!

@lalov1 lalov1 deleted the fix/text-change branch September 6, 2018 13:11
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.

Debugging: Use typeof to Check the Type of a Variable

2 participants