Translate page dont-call-proptypes.md #19
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Matheus Clark dos Santos Monte <matheus.m@samsung.com>
matheusmonte
changed the title
|
#1 Warning -> Don't call Proptypes Warning Page |
glaucia86
reviewed
Feb 8, 2019
glaucia86
reviewed
Feb 8, 2019
glaucia86
reviewed
Feb 8, 2019
glaucia86
reviewed
Feb 8, 2019
glaucia86
reviewed
Feb 8, 2019
| ) | ||
| } | ||
| ``` | ||
|
|
||
| In this case, `ThirdPartyPropTypes.deprecated` is a wrapper calling `PropTypes.bool`. This pattern by itself is fine, but triggers a false positive because React thinks you are calling PropTypes directly. The next section explains how to fix this problem for a library implementing something like `ThirdPartyPropTypes`. If it's not a library you wrote, you can file an issue against it. | ||
| Nesse caso, `ThirdPartyPropTypes.deprecated` é um wrapper chamando `PropTypes.bool`. Esse padrão por si só já é o suficiente, mas dispara um falso positivo pelo fato do React pensar que está chamando diretamente a PropTypes. No próximo tópico, iremos explicar como resolver esse problema para a implementação de uma biblioteca, algo como `ThirdPartyPropTypes`. Caso não seja uma biblioteca que você escreveu, você pode abrir um issue por isso. |
There was a problem hiding this comment.
Suggested change
| Nesse caso, `ThirdPartyPropTypes.deprecated` é um wrapper chamando `PropTypes.bool`. Esse padrão por si só já é o suficiente, mas dispara um falso positivo pelo fato do React pensar que está chamando diretamente a PropTypes. No próximo tópico, iremos explicar como resolver esse problema para a implementação de uma biblioteca, algo como `ThirdPartyPropTypes`. Caso não seja uma biblioteca que você escreveu, você pode abrir um issue por isso. | |
| Nesse caso, `ThirdPartyPropTypes.deprecated` é um wrapper chamando `PropTypes.bool`. Esse padrão por si só já é o suficiente. Mas, dispara um falso positivo pelo fato do React pensar que está chamando diretamente a PropTypes. No próximo tópico, iremos explicar como resolver esse problema para a implementação de uma biblioteca, algo como `ThirdPartyPropTypes`. Caso não seja uma biblioteca que você escreveu, você pode abrir um issue para isso. |
There was a problem hiding this comment.
Acho a modificação de "[...], mas [...]" para "[...]. Mas, [...]" errada neste caso.
glaucia86
reviewed
Feb 8, 2019
glaucia86
reviewed
Feb 8, 2019
|
|
||
| Here is how to fix it. We will use `deprecated` from [react-bootstrap/react-prop-types](https://github.com/react-bootstrap/react-prop-types/blob/0d1cd3a49a93e513325e3258b28a82ce7d38e690/src/deprecated.js) as an example. The current implementation only passes down the `props`, `propName`, and `componentName` arguments: | ||
| Aqui está como corrigir isso. Usaremos `deprecated` daqui [react-bootstrap/react-prop-types](https://github.com/react-bootstrap/react-prop-types/blob/0d1cd3a49a93e513325e3258b28a82ce7d38e690/src/deprecated.js) como exemplo. A atual implementação só passa adiante as `props`, `propName`, and `componentName` arguments: |
There was a problem hiding this comment.
Suggested change
| Aqui está como corrigir isso. Usaremos `deprecated` daqui [react-bootstrap/react-prop-types](https://github.com/react-bootstrap/react-prop-types/blob/0d1cd3a49a93e513325e3258b28a82ce7d38e690/src/deprecated.js) como exemplo. A atual implementação só passa adiante as `props`, `propName`, and `componentName` arguments: | |
| Aqui está como corrigir isso. Usaremos `deprecated` daqui [react-bootstrap/react-prop-types](https://github.com/react-bootstrap/react-prop-types/blob/0d1cd3a49a93e513325e3258b28a82ce7d38e690/src/deprecated.js) como exemplo. A atual implementação só passa adiante as `props`, `propName`, e`componentName` arguments: |
glaucia86
reviewed
Feb 8, 2019
glaucia86
requested changes
Feb 8, 2019
There was a problem hiding this comment.
@matheusmonte algumas alterações a serem corrigidas.
Pedimos que deem uma olhada no review feito e quando for corrigir, pedimos que marque como 'Resolve Conversation'
Após aprovação de 2 code reviews, iremos aprovar a sua contribuição. Muito obrigada! :)
added 2 commits
February 8, 2019 17:35
Signed-off-by: Matheus Clark dos Santos Monte <matheus.m@samsung.com>
Signed-off-by: Matheus Clark dos Santos Monte <matheus.m@samsung.com>
glaucia86
approved these changes
Feb 8, 2019
|
cc: @cezaraugusto could you see if this PR it's okay? |
cezaraugusto
approved these changes
Feb 9, 2019
|
@glaucia86 could you do a follow-up PR with the changes you requested here? I'm merging now |
90 tasks
cezaraugusto
added
🌐 translated
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Signed-off-by: Matheus Clark dos Santos Monte matheus.m@samsung.com