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

(examples/with-i18n-next-intl): fixes warning #25928

Merged

Conversation

akellbl4
Copy link

@akellbl4 akellbl4 commented Jun 9, 2021

Documentation / Examples

Fixed one of the items from #25854

  • Fixes warning from next-intl
  • Hides locale selector in case if no other location
  • Fixes path next/router import
  • Removes unused variable

  • Add the StackBlitz button in README.md
  • Make sure the linting passes

Live demo on StackBlitz

@ijjk ijjk added the examples Issue was opened via the examples template. label Jun 9, 2021
@d3lm
Copy link
Contributor

d3lm commented Jun 9, 2021

@akellbl4 Can you also add the StackBlitz button to the README.md if it works? Here's the snippet you can add:

## Preview

Preview the example live on [StackBlitz](http://stackblitz.com/):

[![Open in StackBlitz](https://developer.stackblitz.com/img/open_in_stackblitz.svg)](https://stackblitz.com/github/vercel/next.js/tree/canary/examples/with-i18n-next-intl)

I have tested it on your fork.

@d3lm d3lm mentioned this pull request Jun 9, 2021
20 tasks
@akellbl4
Copy link
Author

akellbl4 commented Jun 9, 2021

@d3lm as I can see this example already has the StackBlitz button.
https://github.com/vercel/next.js/tree/8347f77f008e2173967303e2b3ae0807a9d475b2/examples/with-i18n-next-intl

@d3lm
Copy link
Contributor

d3lm commented Jun 9, 2021

Ohhh, good catch. Then forget what I said! 😂

@akellbl4 akellbl4 changed the title examples/with-i18n-next-intl: fixes warning (examples/with-i18n-next-intl): fixes warning Jun 11, 2021
Copy link
Member

@ijjk ijjk left a comment

Choose a reason for hiding this comment

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

Thanks for the PR!

@ijjk ijjk requested review from huozhi, padmaia and styfle as code owners July 19, 2021 23:44
@kodiakhq kodiakhq bot merged commit 4e79481 into vercel:canary Jul 19, 2021
flybayer pushed a commit to blitz-js/next.js that referenced this pull request Aug 19, 2021
### Documentation / Examples

Fixed one of the items from vercel#25854

- Fixes warning from `next-intl`
- Hides locale selector in case if no other location
- Fixes path `next/router` import 
- Removes unused variable

---

- [x] Add the StackBlitz button in README.md
- [x] Make sure the linting passes

[Live demo on StackBlitz](https://stackblitz.com/github/akellbl4/next.js/tree/fix/example-with-i18n-next-intl-missing-other/examples/with-i18n-next-intl)
@vercel vercel locked as resolved and limited conversation to collaborators Jan 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
examples Issue was opened via the examples template.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants