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

feat: bump styled-components version on examples #25826

Conversation

Lukazovic
Copy link
Contributor

@Lukazovic Lukazovic commented Jun 6, 2021

Hey guys!
Some time ago I used the with-grommet example to start a project and I ran into some issues because of the styled-components version. I tried to use some new features and it didn't work as I expected, then I noticed the styled-component version was the v4. When I bumped the version it started to work. I am bumping all the styled-components versions on examples because some people may run into the same issue.

Bug

  • Related issues linked using fixes #number
  • Integration tests added

Feature

  • Implements an existing feature request or RFC. Make sure the feature request has been accepted for implementation before opening a PR.
  • Related issues linked using fixes #number
  • Integration tests added
  • Documentation added
  • Telemetry added. In case of a feature if it's used or not.

Documentation / Examples

  • Make sure the linting passes

@ijjk ijjk added the examples Issue/PR related to examples label Jun 6, 2021
Copy link
Member

@leerob leerob left a comment

Choose a reason for hiding this comment

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

Have you validated all of these examples run correctly, given there are breaking changes?

@Lukazovic
Copy link
Contributor Author

Lukazovic commented Jun 6, 2021

Have you validated all of these examples run correctly, given there are breaking changes?

Hey @leerob. Yes, I Have. First I tried to run each one as it was, then I bumped the versions and tried to run each one again (reinstalling the dependencies). I even tested it with disabled javascript to validate the SSR, and it looks good.

Copy link
Member

@leerob leerob left a comment

Choose a reason for hiding this comment

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

Thanks!

@kodiakhq kodiakhq bot merged commit 5879641 into vercel:canary Jun 6, 2021
@Lukazovic Lukazovic deleted the feat/update-styled-components-examples-versions branch June 6, 2021 19:59
flybayer pushed a commit to blitz-js/next.js that referenced this pull request Jun 16, 2021
Hey guys! 
Some time ago I used the `with-grommet` example to start a project and I ran into some issues because of the styled-components version. I tried to use some new features and it didn't work as I expected, then I noticed the styled-component version was the `v4`. When I bumped the version it started to work. I am bumping all the styled-components versions on examples because some people may run into the same issue.

## Bug

- [ ] Related issues linked using `fixes #number`
- [ ] Integration tests added

## Feature

- [ ] Implements an existing feature request or RFC. Make sure the feature request has been accepted for implementation before opening a PR.
- [ ] Related issues linked using `fixes #number`
- [ ] Integration tests added
- [ ] Documentation added
- [ ] Telemetry added. In case of a feature if it's used or not.

## Documentation / Examples

- [x] Make sure the linting passes
@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/PR related to examples
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants