freeCodeCamp / curriculum

The freeCodeCamp curriculum and lesson editor
Creative Commons Attribution Share Alike 4.0 International
81 stars 124 forks source link

fix(challenges): add note at bottom of description in d3 challenge #259

Closed nischaldutt closed 6 years ago

nischaldutt commented 6 years ago

Add a short message in note at bottom of description in d3 challenge

ISSUES CLOSED: https://github.com/freeCodeCamp/freeCodeCamp/issues/17767

Description

Pre-Submission Checklist

Checklist:

Closes https://github.com/freeCodeCamp/freeCodeCamp/issues/17767

scissorsneedfoodtoo commented 6 years ago

@nischaldutt, thank you for taking the time to fix this issue. However, there is a small grammatical problem and an issue with some of the wording in the note you added. It seems that the circles are not really styled in later challenges, at least using the d3 style method. Instead, they just have attributes like radius and height applied to them. Would you mind updating your PR so the note reads something like the following?

"<strong>Note</strong><br>The circles won't be visible because we haven't set their attributes yet. We'll do that in the next challenge."

This is just a suggestion, so let me know what you think about the updated note and if you have any other ideas.

Please take a look at this section of the Contributor's Guide for how to update your PR and amend your git commit message. Also, feel free to reach out if you need any help with the process.

nischaldutt commented 6 years ago

yes sir, I'll look forward to it.

scissorsneedfoodtoo commented 6 years ago

@nischaldutt, alright! Looking forward to your updated description.

nischaldutt commented 6 years ago

thank you sir, I've update the pull request as specified 😊 Do I need to add further description? If so then What do I need to mention? sorry if I am bothering you... its my first time

scissorsneedfoodtoo commented 6 years ago

Yes, would you mind updating the note so it reads something like the following?

"<strong>Note</strong><br>The circles won't be visible because we haven't set their attributes yet. We'll do that in the next challenge."

To do that, make sure you're on the right branch with git checkout add/note-in-description-d3-challenge. Make the changes and add the file with git add challenges/04-data-visualization/data-visualization-with-d3.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 dev and we'll see the changes reflected here.

Let me know if you need any help throughout the process!

nischaldutt commented 6 years ago

Thanks for the valuable response :) I've pushed the commit.

scissorsneedfoodtoo commented 6 years ago

@nischaldutt, I'm not seeing the changes here, and I think it's due to an error in the instructions I gave earlier. I'm very sorry, but I believe I told you to force push to the dev branch rather than the one you created for this PR.

Let's try again:

All you should have to do is make sure you're on the right branch with git checkout add/note-in-description-d3-challenge. Then just run git push -f origin add/note-in-description-d3-challenge. After that we should see the changes you made here.

To get your dev branch back to normal, switch to it with git checkout dev, then run git pull --rebase upstream dev. Finally, all you should have to do is force push to the dev branch with git push -f.

Sorry again for the trouble!

nischaldutt commented 6 years ago

Makes sense, thanks for such a constructive response I've committed the changes

scissorsneedfoodtoo commented 6 years ago

:sparkles: :sparkles: :sparkles: Awesome! Congratulations on your first contribution to freeCodeCamp! :sparkles: :sparkles: :sparkles:

Thank you for submitting a PR for this issue, @nischaldutt. We're all really looking forward to your next contribution! And sorry again for the incorrect directions earlier -- I'll be more careful next time you contribute.

nischaldutt commented 6 years ago

Thanks for guiding me throughout the process. I learned allot from it