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

Translating Components and Props page #24

Merged
merged 24 commits into from Apr 16, 2019

Conversation

helios1101
Copy link
Contributor

@helios1101 helios1101 commented Mar 7, 2019

Translating the Components and Props page to Hindi.
Related to #1
Review Progress

  • Components and Props (@arshadkazmi42 )
  • फंक्शन्स तथा क्लास कौम्पोनॅन्टस (@arshadkazmi42 )
  • कौम्पोनॅन्ट रेंडरिंग (@arshadkazmi42 )
  • कौम्पोनॅन्टस लिखना (@arshadkazmi42 )
  • कौम्पोनॅन्टस निकालना (@arshadkazmi42 )
  • Props केवल-पढ़ने के लिए हैं (@arshadkazmi42 )

@netlify
Copy link

netlify bot commented Mar 7, 2019

Deploy preview for hi-reactjs ready!

Built with commit 385b535

https://deploy-preview-24--hi-reactjs.netlify.com

@helios1101 helios1101 changed the title Adding Components and Props page Translating Components and Props page Mar 8, 2019
Copy link
Member

@arshadkazmi42 arshadkazmi42 left a comment

Choose a reason for hiding this comment

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

Good start.
I have left a few feedbacks.
Also do have a read of our review process at #23

content/docs/components-and-props.md Outdated Show resolved Hide resolved
content/docs/components-and-props.md Outdated Show resolved Hide resolved
content/docs/components-and-props.md Outdated Show resolved Hide resolved
content/docs/components-and-props.md Outdated Show resolved Hide resolved
content/docs/components-and-props.md Outdated Show resolved Hide resolved
content/docs/components-and-props.md Outdated Show resolved Hide resolved
content/docs/components-and-props.md Outdated Show resolved Hide resolved
content/docs/components-and-props.md Outdated Show resolved Hide resolved
content/docs/components-and-props.md Outdated Show resolved Hide resolved
content/docs/components-and-props.md Outdated Show resolved Hide resolved
@arshadkazmi42
Copy link
Member

I have done the review till line 90, once these are fixed, we will continue the review of next parts

@arshadkazmi42
Copy link
Member

@helios1101 Also in PR description can you add list of all headings similar to how you have added page name

content/docs/components-and-props.md Outdated Show resolved Hide resolved
content/docs/components-and-props.md Outdated Show resolved Hide resolved
content/docs/components-and-props.md Outdated Show resolved Hide resolved
content/docs/components-and-props.md Outdated Show resolved Hide resolved
Copy link
Member

@arshadkazmi42 arshadkazmi42 left a comment

Choose a reason for hiding this comment

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

Great work.
Added few more and which I missed initially.
and you have missed 1 fix.
Once these are done, we will continue with next parts of review

Copy link
Member

@arshadkazmi42 arshadkazmi42 left a comment

Choose a reason for hiding this comment

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

Awesome work.
I have added some more feedbacks for rest of the file.

content/docs/components-and-props.md Outdated Show resolved Hide resolved
content/docs/components-and-props.md Outdated Show resolved Hide resolved
content/docs/components-and-props.md Outdated Show resolved Hide resolved
content/docs/components-and-props.md Outdated Show resolved Hide resolved
content/docs/components-and-props.md Outdated Show resolved Hide resolved
content/docs/components-and-props.md Outdated Show resolved Hide resolved
content/docs/components-and-props.md Outdated Show resolved Hide resolved
content/docs/components-and-props.md Outdated Show resolved Hide resolved

```js
function sum(a, b) {
return a + b;
}
```

Such functions are called ["pure"](https://en.wikipedia.org/wiki/Pure_function) because they do not attempt to change their inputs, and always return the same result for the same inputs.
यह फंक्शन्स को हम कहते है ["pure"](https://en.wikipedia.org/wiki/Pure_function) क्योंकि वे अपने इनपुट को बदलने का प्रयास नहीं करते हैं, और हमेशा समान इनपुट के लिए समान परिणाम लौटाते हैं|
Copy link
Member

Choose a reason for hiding this comment

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

यह => ऐसे

Copy link
Member

Choose a reason for hiding this comment

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

You missed this one

यह => ऐसे

content/docs/components-and-props.md Outdated Show resolved Hide resolved
Copy link
Member

@arshadkazmi42 arshadkazmi42 left a comment

Choose a reason for hiding this comment

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

Awesome.
I missed a couple of things in previous review, my bad.
I have pointed those out, once you are done with those changes, it will be done from my end and we can move on to second phase of the review

content/docs/components-and-props.md Outdated Show resolved Hide resolved

```js
function sum(a, b) {
return a + b;
}
```

Such functions are called ["pure"](https://en.wikipedia.org/wiki/Pure_function) because they do not attempt to change their inputs, and always return the same result for the same inputs.
यह फंक्शन्स को हम कहते है ["pure"](https://en.wikipedia.org/wiki/Pure_function) क्योंकि वे अपने इनपुट को बदलने का प्रयास नहीं करते हैं, और हमेशा समान इनपुट के लिए समान परिणाम लौटाते हैं|
Copy link
Member

Choose a reason for hiding this comment

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

You missed this one

यह => ऐसे

content/docs/components-and-props.md Outdated Show resolved Hide resolved
content/docs/components-and-props.md Outdated Show resolved Hide resolved
Copy link
Member

@arshadkazmi42 arshadkazmi42 left a comment

Choose a reason for hiding this comment

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

Great work. Thank you for working on this and fixing all the review feedback.
Review is done from me.
For second review @saranshkataria can you have a look at this for the second review


We recommend naming props from the component's own point of view rather than the context in which it is being used.
Props का नामकरण हम कौम्पोनॅन्टस के अनुसार करने की सलाह देते हैं बजाय उस संदर्भ में किया जाए जिसमें इसका उपयोग किया जा रहा है|
Copy link
Member

Choose a reason for hiding this comment

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

can we break this into easier wording instead?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Couldn't find simple words.
I can just reverse the parts ...will that do ??

Copy link
Member

Choose a reason for hiding this comment

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

बजाय उस संदर्भ में किया जाए जिसमें इसका उपयोग किया जा रहा है| part is just a bit difficult, could we reword that and split it into 2 sentences that way?

@saranshkataria
Copy link
Member

Done till {#props-are-read-only}. 1 more section to go after this. So close 👍

@saranshkataria
Copy link
Member

added final set of comments. Almost there 🎆

@saranshkataria
Copy link
Member

@helios1101 any updates on this?

@helios1101
Copy link
Contributor Author

@helios1101 any updates on this?

I'm having my college sessional exams, will revert once it gets over.

@saranshkataria
Copy link
Member

@helios1101 any updates on this?

I'm having my college sessional exams, will revert once it gets over.

no worries. Good luck!

@helios1101
Copy link
Contributor Author

I'm having my college sessional exams, will revert once it gets over.

no worries. Good luck!

@saranshkataria Plz have a look at the commit!

@saranshkataria
Copy link
Member

awesome work! Thanks!

@saranshkataria saranshkataria merged commit 0d25cb2 into reactjs:master Apr 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants