Skip to content

Conversation

@LucasFlaquer
Copy link

Proposed Solution

file translation #10 "guide/forms.md"

Additional Information

I've got doubts about translating the codepe's link so I left the way it was

@ErickPetru ErickPetru changed the title Tradução do arquivo "guide/forms.md" #10 docs: translate file "guide/forms.md", closes #10 Sep 17, 2020
@ErickPetru ErickPetru self-requested a review September 17, 2020 17:21

::: tip Note
::: tip Nota
`v-model` will ignore the initial `value`, `checked` or `selected` attributes found on any form elements. It will always treat the current active instance data as the source of truth. You should declare the initial value on the JavaScript side, inside the `data` option of your component.
Copy link
Member

Choose a reason for hiding this comment

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

As dicas e avisos também precisam ser traduzidos.

Copy link
Author

Choose a reason for hiding this comment

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

Eu realizei mais um commit atualizando esta parte. Preciso abrir outro pull Request ou é possível pegar as alterações a partir deste?

Copy link
Member

Choose a reason for hiding this comment

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

Não precisa, os commits são incrementais dentro do mesmo pull request.

<p>A mensagem é: {{ message }}</p>
```

<p class="codepen" data-height="300" data-theme-id="39028" data-default-tab="result" data-user="Vue" data-slug-hash="eYNPEqj" data-editable="true" style="height: 300px; box-sizing: border-box; display: flex; align-items: center; justify-content: center; border: 2px solid; margin: 1em 0; padding: 1em;" data-pen-title="Handling forms: basic v-model">
Copy link
Member

Choose a reason for hiding this comment

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

Na doc V2 chegamos a fazer fork e traduzir muitos CodePens. Seria legal conferir se os exemplos ainda são os mesmos e aproveitar os links da V2.

Copy link
Author

Choose a reason for hiding this comment

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

@ErickPetru procurei os pens em português mas não os encontrei. Poderia me ajudar?

Copy link
Member

Choose a reason for hiding this comment

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

Este é arquivo traduzido da V2: https://github.com/vuejs-br/br.vuejs.org/blob/master/src/v2/guide/forms.md. Você pode dar uma olhada nos exemplos e ver se mudaram ou não.

Copy link
Author

Choose a reason for hiding this comment

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

Obrigado. Eu andei lendo outros prs e vi que você comentou de fazer um fork para o codepen do vuejs-br então estou criando no meu codepen para depois serem feitos os forks. Assim que eu terminar aviso para poder colocar o link vindo do codepen do vuejs-br

LucasFlaquer and others added 23 commits September 17, 2020 21:15
* Note that props can be accessed in template directly In "Usage with Templates" the doc shows how props of the object returned from `setup` can be used directly in the template, but I didn't find anywhere else in the composition-api doc where it says that props can also be used directly. This seemed like as good a place as any to add that, since that's where I was looking for it first :-). * Update src/guide/composition-api-setup.md Co-authored-by: Natalia Tepluhina <tarya.se@gmail.com>
Co-authored-by: Ben Hong <ben@bencodezen.io>
* config: add dev command * feature: standardize video styles and markup
The link seems to have been broken with commit 7e55600 ([pretty GH view][pretty-view]). [pretty-view]: vuejs@7e55600#diff-c9868b6bf75c863c7384dcf155eb6497R35
* Update v-bind.md * change property to properties Co-authored-by: Kid <44045911+kidonng@users.noreply.github.com> Co-authored-by: Kid <44045911+kidonng@users.noreply.github.com>
CyberAP and others added 9 commits September 19, 2020 15:41
* Add usage examples for Asset Options * Update src/api/options-assets.md Co-authored-by: Natalia Tepluhina <tarya.se@gmail.com> * Update src/api/options-assets.md Co-authored-by: Natalia Tepluhina <tarya.se@gmail.com> Co-authored-by: Natalia Tepluhina <tarya.se@gmail.com>
…attributes, add the `onError` event, and attach a comment (vuejs#458)
@ErickPetru
Copy link
Member

Olá @LucasFlaquer, aqui está a coleção com os pens adicionados à conta vuejs-br: https://codepen.io/collection/XPEEqp.
Por favor, atualize sua tradução com estes links.

Além disso, por favor, desfaça os commits relacionados a merge de upstream, apenas a equipe de revisão faz merge de upstream diretamente neste repositório, senão os conflitos de merge e possíveis sobrescritas ficam descontrolados.

Ou seja, você deve enviar neste pull request apenas o arquivo a que ele se propõe: "guide/forms.md".

@ErickPetru ErickPetru marked this pull request as draft September 19, 2020 13:45
@LucasFlaquer
Copy link
Author

olá @ErickPetru desculpe pelo problema com o git quis baixar as ultimas altualizações. Não sabia que iria afetar este PR. Já atualizei os links

@LucasFlaquer LucasFlaquer marked this pull request as ready for review September 22, 2020 18:23
@ErickPetru
Copy link
Member

Olá @LucasFlaquer, ainda tá com problema por causa do merge. Para evitar mais confusões com PR e commit, se preferir guardar seu arquivo traduzido por aí, fechar este PR e enviar um novo só com o arquivo traduzido, não tem problema.

GuiDevloper pushed a commit that referenced this pull request Aug 29, 2021
axios was added in #31, but is now unused as code examples were moved to CodePen in #57. Let's remove axios.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet