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: backdrop stage abstraction #116

Merged
merged 6 commits into from
Jul 12, 2023
Merged

Conversation

alvarosabu
Copy link
Member

No description provided.

@netlify
Copy link

netlify bot commented Jul 1, 2023

Deploy Preview for cientos-tresjs ready!

Name Link
🔨 Latest commit 95c4c7c
🔍 Latest deploy log https://app.netlify.com/sites/cientos-tresjs/deploys/64aeef735ec9290008a769d5
😎 Deploy Preview https://deploy-preview-116--cientos-tresjs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Member

Choose a reason for hiding this comment

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

Why do we add models and textures here in cientos?

Should not go in assets repo?

Copy link
Member Author

Choose a reason for hiding this comment

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

I though for docs would be ok to have them locally for performance, but seems like that's breaking when statically generating the assets.

Copy link
Member

Choose a reason for hiding this comment

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

Same here, is there a reason why these textures are here?

Should not go in assets?

@JaimeTorrealba
Copy link
Member

I check the example in netlify deploy, but I'm not sure if is right.

https://deploy-preview-116--cientos-tresjs.netlify.app/guide/staging/backdrop.html

image

@alvarosabu alvarosabu merged commit 62f677c into main Jul 12, 2023
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