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

PlaneBufferGeometry has been renamed to PlaneGeometry. #640

Open
wants to merge 3 commits into
base: dev
Choose a base branch
from

Conversation

ma2yama
Copy link

@ma2yama ma2yama commented Jan 9, 2025

⚠️ All PRs have to be done versus 'dev' branch, so be aware of that, or we'll close your issue ⚠️

What kind of change does this PR introduce?

Fixes to avoid using the deprecated API.

Can it be referenced to an Issue? If so what is the issue # ?

No

How can we test it?

Use any of location-based examples with the modified code.

Summary

Since the API has been removed from Three.js, it has been modified to use a different API.

Does this PR introduce a breaking change?

No

Please TEST your PR before proposing it. Specify here what device you have used for tests, version of OS and version of Browser

iPhone 14 Pro, mobile Safari, iOS 17.7

Other information

mrdoob/three.js#26175

dependabot bot and others added 3 commits September 30, 2023 22:08
Bumps [yaml](https://github.com/eemeli/yaml) from 2.2.1 to 2.3.2.
- [Release notes](https://github.com/eemeli/yaml/releases)
- [Commits](eemeli/yaml@v2.2.1...v2.3.2)

---
updated-dependencies:
- dependency-name: yaml
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
changed Twitter's old name to new one 𝕏
@nickw1
Copy link
Collaborator

nickw1 commented Jan 9, 2025

@ma2yama many thanks, looks good: let me test it. We have already fixed this issue in LocAR.js (see: https://github.com/AR-js-org/locar.js/blob/master/lib/src/webcam-renderer.js); however it hasn't been fixed in the main repo yet.

One request, could you re-open your PR against dev? All PRs get merged into dev branch, which is then merged into master when we make a release. Thanks.

@kalwalt
Copy link
Member

kalwalt commented Jan 9, 2025

For some reason these changes wasn't effective, probably during the merge process something went wrong. Thank you @ma2yama to note this.

@kalwalt kalwalt added bug Something isn't working enhancement New feature or request location based labels Jan 9, 2025
@nickw1
Copy link
Collaborator

nickw1 commented Jan 9, 2025

@kalwalt yes, I did initially think that this had been updated already in the last release.

@kalwalt kalwalt changed the base branch from master to dev January 18, 2025 22:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request location based
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants