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

fix(types): add typing for Routes meta #89

Merged
merged 1 commit into from
Sep 30, 2019
Merged

fix(types): add typing for Routes meta #89

merged 1 commit into from
Sep 30, 2019

Conversation

juliomrqz
Copy link
Contributor

Add support for declaring the meta object in Nuxt.js Page, as we can see here https://github.com/nuxt/nuxt.js/tree/dev/examples/routes-meta 🤓 .

Add support for declaring the meta object in Nuxt.js Page, as we can see here https://github.com/nuxt/nuxt.js/tree/dev/examples/routes-meta 🤓 .
@kevinmarrec kevinmarrec added the fix Pull requests that fix a bug label Aug 28, 2019
@kevinmarrec
Copy link
Contributor

kevinmarrec commented Aug 29, 2019

@juliomrqz Thanks !

We're working on changing/fixing some implementation around the route meta, and also document it on nuxtjs.org.

In fact, this feature never has been documented, it only has been showcased around the example you linked.

We agreed with other core team members to wait for these changes before providing types for this feature. Thanks in advance for your understanding 🙂

@juliomrqz
Copy link
Contributor Author

@kevinmarrec no problem, I understand

@kevinmarrec kevinmarrec added the types The issue or pull request is related to `types` package label Aug 30, 2019
@kevinmarrec
Copy link
Contributor

@juliomrqz Finally it sounds good to me so I'm merging it right now

@kevinmarrec kevinmarrec merged commit 19df2f9 into nuxt:master Sep 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix Pull requests that fix a bug types The issue or pull request is related to `types` package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants