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

Spelling error in Basic Node and Express - Get Route Parameter Input from the Client #17991

Closed
sheikheddy opened this issue Aug 8, 2018 · 5 comments · Fixed by freeCodeCamp/curriculum#244
Labels
first timers only Open for anyone doing contributions for first time. You do not need permission to work on these.

Comments

@sheikheddy
Copy link

Describe your problem and - if possible - how to reproduce it

it says 'communicate us' instead of 'communicate to us'
How to reproduce it? Just go to the page :P

Add a Link to the page with the problem

https://learn.freecodecamp.org/apis-and-microservices/basic-node-and-express/get-route-parameter-input-from-the-client

Tell us about your browser and operating system

  • Browser Name: Chrome
  • Browser Version: 67.0.3396.99
  • Operating System: Windows 10

If possible, add a screenshot here

image

@moT01
Copy link
Member

moT01 commented Aug 8, 2018

its missing an m in communicate as well, - this is where the problem is => https://github.com/freeCodeCamp/curriculum/blob/dev/challenges/05-apis-and-microservices/basic-node-and-express.json#L190

@moT01 moT01 added the first timers only Open for anyone doing contributions for first time. You do not need permission to work on these. label Aug 8, 2018
@RyanPisuena
Copy link
Contributor

RyanPisuena commented Aug 8, 2018

Hello all parties involved,

Below is a pull request for the grammar edit for you two.

freeCodeCamp/curriculum#227

@tom-raley
Copy link
Contributor

Is this issue still needing to be solved? I'd like to work on it if so!

@fatimah5
Copy link

This is my first contribution, I would like to try to fix this problem by replacing the word
"comunicte us" to "communicate us".

Thank you.

@krbrennan
Copy link

This is a duplicate of issue #244. I submitted changes to this one but some other zealot fixed 50 typos so this current issue is either fixed or will be fixed very shortly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
first timers only Open for anyone doing contributions for first time. You do not need permission to work on these.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants