Skip to content
This repository was archived by the owner on Sep 3, 2024. It is now read-only.

Add/fix information on nuxtChildKey #1331

Merged
merged 3 commits into from
May 10, 2019
Merged

Add/fix information on nuxtChildKey #1331

merged 3 commits into from
May 10, 2019

Conversation

hatashiro
Copy link
Contributor

This is the document-side pull request of nuxt/nuxt#5516.

I've updated documentation on nuxtChildKey prop.

  1. The default is not $route.fullPath. $route.path is used by default.
  2. Documentation to handle internal key of <router-view/> is added.

FYI, I did the task 1 and 2 for EN, DE, ES, JA and KO.

  • EN, DE, ES: They were written yet in English, so I just wrote in EN too.
  • JA, KO: I can write JA and KO.

For the other languages, I did the task 1 because fullPath by default is wrong information and better be fixed soon.

cc: @potato4d and @DiyLecko

Hyunje Jun added 2 commits April 24, 2019 17:19
@clarkdo
Copy link
Member

clarkdo commented Apr 24, 2019

@atinux Base branch should be changed since the pr is a breaking change.

@atinux
Copy link
Member

atinux commented May 6, 2019

Well actually the documentation was wrong and the default value was not $route.fullPath 🙈

@atinux
Copy link
Member

atinux commented May 6, 2019

@inouetakuya could you please resolve the conflicts for ja/api/components-nuxt.md please 🙏 ?

@atinux atinux requested a review from inouetakuya May 6, 2019 09:58
@potato4d
Copy link
Collaborator

potato4d commented May 6, 2019

@atinux @inouetakuya
I noticed this PR now.
It can be solved by me!

@atinux
Copy link
Member

atinux commented May 6, 2019

Thanks @potato4d :)

@clarkdo clarkdo changed the base branch from master to 2.7 May 7, 2019 09:09
@inouetakuya
Copy link
Collaborator

inouetakuya commented May 9, 2019

@atinux @potato4d

could you please resolve the conflicts for ja/api/components-nuxt.md please 🙏 ?

done 😄

But I have no idea to merge this PR or not. So please handle it 🙏

@potato4d
Copy link
Collaborator

potato4d commented May 9, 2019

@inouetakuya
Thank you for doing it. sorry 🙇

@atinux atinux merged commit 1b36c40 into nuxt:2.7 May 10, 2019
@@ -10,7 +10,7 @@ description: 该组件用于在布局中显示页面组件(即非布局内容
**Props**:
- nuxtChildKey: `string`
- 此prop将设置为`<router-view />`,可用于在动态页面和不同路径内进行转换。
- 默认: `$route.fullPath`
- 默认: `$route.path`
Copy link
Member

Choose a reason for hiding this comment

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

@warriorBrian Could you please help to add Chinese translation when you have time ?

@atinux atinux mentioned this pull request May 10, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants