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

Update with-three-js example #24857

Merged
merged 4 commits into from
May 10, 2021
Merged

Conversation

HaNdTriX
Copy link
Contributor

@HaNdTriX HaNdTriX commented May 6, 2021

Changes

  • Update dependencies
  • Use new useAnimations hook
  • Remove next-transpile-modules in favour of drei
  • Refactor Components
  • Remove dynamic import
  • Enable webpack5 (by removing next.config.js)
  • Removed missing key warning

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 was opened via the examples template. label May 6, 2021
@HaNdTriX HaNdTriX force-pushed the examples/three-js branch from 3950ed4 to 7d6abc6 Compare May 6, 2021 11:03
leerob
leerob previously approved these changes May 6, 2021
@kodiakhq kodiakhq bot merged commit a18c3c2 into vercel:canary May 10, 2021
flybayer pushed a commit to blitz-js/next.js that referenced this pull request Jun 1, 2021
## Changes

- Update dependencies
- Use new `useAnimations` hook
- Remove next-transpile-modules in favour of drei
- Refactor Components
- Remove dynamic import
- Enable webpack5 (by removing `next.config.js`)
- Removed missing key warning

## 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 was opened via the examples template.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants