Skip to content

Added postinstall script that updates the three importmap version#4260

Merged
elalish merged 2 commits intogoogle:masterfrom
Beilinson:master
Apr 28, 2023
Merged

Added postinstall script that updates the three importmap version#4260
elalish merged 2 commits intogoogle:masterfrom
Beilinson:master

Conversation

@Beilinson
Copy link
Contributor

Whenever npm install is run in packages/model-viewer, packages/model-viewer-effects/README.md and the examples in packages/modelviewer.dev/examples/postprocessing/index.html are automatically updated to be in sync with the three version in package.json.

@Beilinson Beilinson mentioned this pull request Apr 28, 2023
@elalish elalish merged commit 0f55b43 into google:master Apr 28, 2023
@elalish
Copy link
Contributor

elalish commented Apr 28, 2023

We're starting to get requests already for post-processing in AR - would you like to take a look at hooking it up in WebXR? I'm hoping it shouldn't take too much.

@Beilinson
Copy link
Contributor Author

I'll open a separate issue for it with the information I wrote in the original pull request, I'm not sure I'll be able to test it however as I only have access to an Iphone which only experimentally supports WebXR, it would be preferable to test on android.

@elalish elalish mentioned this pull request Jul 14, 2023
JL-Vidinoti pushed a commit to vidinoti/model-viewer that referenced this pull request Apr 22, 2024
…ogle#4260)

* Added postinstall script that updates the three importmap version

* removed commented code
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants