Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(curriculum): Improve verbiage and add examples to the map, filter, and reduce challenges #37709

Conversation

RandellDawson
Copy link
Member

@RandellDawson RandellDawson commented Nov 7, 2019

  • 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 master branch of freeCodeCamp.
  • None of my changes are plagiarized from another source without proper attribution.
  • All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
  • My changes do not use shortened URLs or affiliate links.

Closes #37704

This PR attempts to explain in detail how each of the methods introduced in the following challenges work and includes examples of how to use them. The current versions of these challenges almost assume the user has already been taught how to use them, which is why the forum gets a lot of questions for these challenges.

I also formatted the challenge seed code to use two space indentation, to reduce the line width of the code and make it more readable.

@gitpod-io
Copy link

gitpod-io bot commented Nov 7, 2019

@RandellDawson RandellDawson changed the title fix: improve verbiage and add examples to challenges fix(curriculum): Improve verbiage and add examples to the map, filter, and reduce challenges Nov 7, 2019
@RandellDawson RandellDawson added language: English scope: curriculum Lessons, Challenges, Projects and other Curricular Content in curriculum directory. labels Nov 7, 2019
@RandellDawson RandellDawson force-pushed the fix/add-examples-to-map-filter-reduce-challenges branch from 2178f41 to f3a8cd8 Compare November 7, 2019 05:39
@RandellDawson RandellDawson force-pushed the fix/add-examples-to-map-filter-reduce-challenges branch from f3a8cd8 to 608da40 Compare November 7, 2019 06:20
@RandellDawson
Copy link
Member Author

@raisedadead Why does Gitpod still show Pending status? When I click on Details link, it opens up Gitpod and creates the build and eventually shows everything working as it should.

Copy link
Contributor

@ojeytonwilliams ojeytonwilliams left a comment

Choose a reason for hiding this comment

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

@RandellDawson since you wanted the verbiage shrinking I ended up being quite aggressive with the cuts/re-wording. Let me know if I went too far.

RandellDawson and others added 11 commits November 11, 2019 09:35
Co-Authored-By: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>
Co-Authored-By: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>
Co-Authored-By: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>
Co-Authored-By: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>
Co-Authored-By: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>
Co-Authored-By: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>
Co-Authored-By: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>
Co-Authored-By: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>
Co-Authored-By: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>
Co-Authored-By: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>
Co-Authored-By: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>
Copy link
Contributor

@ojeytonwilliams ojeytonwilliams left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@raisedadead
Copy link
Member

Why does Gitpod still show Pending status? When I click on Details link, it opens up Gitpod and creates the build and eventually shows everything working as it should.

Maybe it's just a glitch or something. 🤷‍♂

Copy link
Contributor

@Manish-Giri Manish-Giri left a comment

Choose a reason for hiding this comment

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

Changes for fixing a few bugs and some improvements.

RandellDawson and others added 5 commits November 11, 2019 23:57
Co-Authored-By: Manish Giri <manish.giri.me@gmail.com>
Co-Authored-By: Manish Giri <manish.giri.me@gmail.com>
Co-Authored-By: Manish Giri <manish.giri.me@gmail.com>
Co-Authored-By: Parth Parth <34807532+thecodingaviator@users.noreply.github.com>
Co-Authored-By: Manish Giri <manish.giri.me@gmail.com>
Copy link
Member

@moT01 moT01 left a comment

Choose a reason for hiding this comment

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

I'm fine with these changes. I think it's a big improvement. Looks good @RandellDawson 🎉

@RandellDawson
Copy link
Member Author

@moT01 I am still working on add @Manish-Giri's suggested changes. I will try to do that today and then request one last look from you all.

@RandellDawson
Copy link
Member Author

@Manish-Giri and @thecodingaviator Can you please take one last review to see if this is ready to merge? Thanks.

Copy link
Contributor

@Manish-Giri Manish-Giri left a comment

Choose a reason for hiding this comment

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

@RandellDawson Looks good. Good job 💯

@thecodingaviator thecodingaviator merged commit b794908 into freeCodeCamp:master Nov 27, 2019
@thecodingaviator
Copy link
Contributor

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

@RandellDawson RandellDawson deleted the fix/add-examples-to-map-filter-reduce-challenges branch November 27, 2019 02:16
abbathaw pushed a commit to abbathaw/freeCodeCamp that referenced this pull request Jul 24, 2020
…, and reduce challenges (freeCodeCamp#37709)

* fix: improve verbiage and add examples to challenges

* fix: condensed map method wording

Co-Authored-By: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>

* fix: remove leading space

Co-Authored-By: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>

* fix: improved verbiage

Co-Authored-By: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>

* fix: shortened reduce challenge verbiage

Co-Authored-By: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>

* fix: shorten verbiage related to reduce 2nd parameter

Co-Authored-By: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>

* fix: added comma

Co-Authored-By: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>

* fix: shortened filter verbiage

Co-Authored-By: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>

* fix; added comma

Co-Authored-By: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>

* fix: improved verbiage of map method

Co-Authored-By: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>

* fix: shortened reduce verbiage

Co-Authored-By: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>

* fix: correct map method verbiage

Co-Authored-By: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>

* fix: added word array

Co-Authored-By: Manish Giri <manish.giri.me@gmail.com>

* fix: added missing comma

Co-Authored-By: Manish Giri <manish.giri.me@gmail.com>

* fix: changed u to sumOfAges

Co-Authored-By: Manish Giri <manish.giri.me@gmail.com>

* fix: created one sentence out of many

Co-Authored-By: Parth Parth <34807532+thecodingaviator@users.noreply.github.com>

* fix: changed to on each iteration

Co-Authored-By: Manish Giri <manish.giri.me@gmail.com>

* fix: fixed verbiage in filter and map challenges
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope: curriculum Lessons, Challenges, Projects and other Curricular Content in curriculum directory.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Javascript map, filter, and reduce methods need examples and explanation of using callback functions
6 participants